logging-flume icon indicating copy to clipboard operation
logging-flume copied to clipboard

fix "eventCounter" increase bug in append()

Open hangc0276 opened this issue 9 years ago • 1 comments

In append() function, when the "batchCounter" is equal to batchSize, flush() will be called, which may throw exceptions when the network jitter happens on HDFS. Once exception happens, the event appended to the batch will be rollback, but the "eventCounter" has already been increased, which will make the "eventCounter" wrong value. So, the "eventCounter" should be increased after flush() happens.

hangc0276 avatar Jan 11 '17 09:01 hangc0276

Can one of the admins verify this patch?

asfgit avatar Aug 17 '18 13:08 asfgit