[GitHub] [incubator-pinot] Jackie-Jiang commented on issue #4339: Only set Helix cluster properties when creating the cluster

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

Jackie-Jiang commented on issue #4339: Only set Helix cluster properties when creating the cluster
URL: https://github.com/apache/incubator-pinot/pull/4339#issuecomment-503398156
 
 
   > Should we make `maxParallelStateTransition` configurable?
   > 
   > @jackjlli Can you comment if there was a particular reason that you set this to 1?
   
   We can make it configurable, but in most of the cases Helix default (40) should be good enough.
   The 1 is there in the legacy code. But before Jack's change, the 1 is applied only when we create a new cluster.

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


With regards,
Apache Git Services

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