[GitHub] [druid] jihoonson commented on pull request #10284: Add dynamic coordinator config that allows control over how many segments are considered when picking a segment to move.

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

jihoonson commented on pull request #10284:
URL: https://github.com/apache/druid/pull/10284#issuecomment-679265465


   > I recently became a committer, but haven't gotten repo access setup yet (If another committer reads this and is willing to guide me on this process, I'm all ears!) so I cannot complete all of the committer steps. For labels, I would add:
   > 
   >     * Design Review - It does add a config and change the balancing strategy interface so a design review seems necessary/smart
   > 
   >     * Feature
   > 
   >     * Performance (I did benchmark this in a dev cluster and we will see shaved time in the pickSegmentToMove code, but it will likely only be noticeable/worthwhile at very large scale)
   > 
   >     * Area - Segment Balancing/Coordination
   
   Hey @capistrant, have you finished the setup for GitBox? You can check it here: https://gitbox.apache.org/setup/.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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