Apache Thrift warning "No seqid to unset" (NodeJS)

Question

"[WARN] No seqid to unset" What are the reasons for this warning? (NodeJS)

score 2 · Answer 1 · answered Jun 11 '18 at 11:16

for version 0.11.0, it's a bug

TBinaryProtocol.prototype.writeMessageEnd = function() {
    if (this._seqid) {
        this._seqid = null;
    } else {
        log.warning('No seqid to unset');
    }
};

this._seqid would false when it is 0.

It's now fixed in branch master

// from `if (this._seqid)` to `if (this._seqid !== null )`
TBinaryProtocol.prototype.writeMessageEnd = function() {
    if (this._seqid !== null) {
        this._seqid = null;
    } else {
        log.warning('No seqid to unset');
    }
};

see source file : https://github.com/apache/thrift/blob/master/lib/nodejs/lib/thrift/binary_protocol.js

score 1 · Accepted Answer · answered Mar 03 '18 at 13:30

1

The reason was a version mismatch. 0.9.3 & latest

answered Mar 03 '18 at 13:30

Aleksandr Maybach

610
8
20

Apache Thrift warning "No seqid to unset" (NodeJS)

2 Answers2