fluent / fluent-logger-node

A structured logger for Fluentd (Node.js)
Apache License 2.0
259 stars 83 forks source link

in non-message mode, queue size seems to grow indefinitely #98

Closed theogravity closed 6 years ago

theogravity commented 6 years ago

I'm looking at the memory profile using this module, and I noticed that this._sendQueueSize never gets reset to 0 after send operations.

https://github.com/fluent/fluent-logger-node/blob/a2fcc64e0fef21a2d2fc286fa82964aa4bccf644/lib/sender.js#L42

Is this intentional? Shouldn't it decrement or reset to 0 during certain conditions?

okkez commented 6 years ago

Thank you for reporting. It is an unexpected behavior. We must reset to 0 after flush sendQueue.

okkez commented 6 years ago

Released v2.8.0