ifndef-SleePy commented on PR #5170:
URL: https://github.com/apache/inlong/pull/5170#issuecomment-1198142926

   After an offline communication with @yunqingmoswu , we reached an agreement 
that just keep the origin implementation with thread-safe counter. Because it's 
bases on production requirement, not technical.
   1. We have to keep the meaningless "dirty" prefix counter till someday the 
production manager realizes that it's meaningless.
   2. Summing the counter after flushing is better than before flushing 
although neither of them could guarantee accuracy. Because we don't want to 
meet the scenario that user asks a question why there exists successful counter 
in metric system however there is no data in HBase.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to