[GitHub] [helix] desaikomal commented on pull request #2463: Initial commit for capacity based throttler for state transition control

["desaikomal (via GitHub)" <gi...@apache.org>]

desaikomal commented on PR #2463:
URL: https://github.com/apache/helix/pull/2463#issuecomment-1537523984

   @junkaixue , @jiajunwang , @NealSun96 - can you please look at the design proposal and let us know your view?
   I do like the separation of concern approach where BestPossibleAssignment does what is the right assignment and we have another stage which does the "filtering" / "throttlng" based on the current state. One stage is doing pure computation while other is trying to take us towards that ideal computed state and doing right "ordering". If you guys think, we need to introduce one additional stage in the pipeline can help, we can think about that too.
   But let us close on this please.
   thanks,


-- 
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: reviews-unsubscribe@helix.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org