[GitHub] clintropolis opened a new pull request #6032: Fix coordinator balancer 'moved' logs and metrics

[GitBox <gi...@apache.org>]

clintropolis opened a new pull request #6032: Fix coordinator balancer 'moved' logs and metrics
URL: https://github.com/apache/incubator-druid/pull/6032
 
 
   Fixes a cosmetic issue where the incorrect 'movedCount' metric and log messaging because the value was sourced from `currentlyMovingSegments.size()` rather than the total count of movement operations done, which could be inaccurate if the load finishes before the balancer operation completes and emits metrics and logs. 
   
   This changes metrics and logs to use the counter value rather than the snapshot of currently moving segments at emit time.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@druid.apache.org
For additional commands, e-mail: dev-help@druid.apache.org