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/10/01 20:20:42 UTC

[GitHub] [helix] xyuanlu commented on pull request #1362: Move ST message to state validation from executing phase to scheduling phase.

xyuanlu commented on pull request #1362:
URL: https://github.com/apache/helix/pull/1362#issuecomment-702375221


   This PR is ready to be merged.
   
   Final commit message:
   Move ST message to state validation from executing phase to scheduling phase.
   
   Move ST message to state validation to scheduling phase.because of the following reasons:
   It is a no-op operation, thus we should not create a thread to execute it.
   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