You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@helix.apache.org by GitBox <gi...@apache.org> on 2020/09/14 18:24:12 UTC

[GitHub] [helix] xyuanlu opened a new issue #1361: Move ST message to state validation from executing phase to scheduling phase.

xyuanlu opened a new issue #1361:
URL: https://github.com/apache/helix/issues/1361


   Currently we do all the state validation (the from state should be same current state and to state should be different to current state) for state transition message when the message is being executed in HelixStateTransitionHandler.
   
   We should move the to state validation to the scheduling stage because of the following reasons:
   1. it is a no-op operation, thus we should not create a thread to execute it.
   2.The lag between scheduling and executing could add additional latency for controller and eventually application latency. 
   
   
   
   


----------------------------------------------------------------
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: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org


[GitHub] [helix] jiajunwang closed issue #1361: Move ST message to state validation from executing phase to scheduling phase.

Posted by GitBox <gi...@apache.org>.
jiajunwang closed issue #1361:
URL: https://github.com/apache/helix/issues/1361


   


----------------------------------------------------------------
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: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org