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/06/01 21:12:38 UTC
[GitHub] [helix] zhangmeng916 opened a new issue #1046: Improve the adding of customized state listener
zhangmeng916 opened a new issue #1046:
URL: https://github.com/apache/helix/issues/1046
In current customized view aggregation implementation, we always add listener to CUSTOMIZED_STATE path for all participants no matter whether the path exists or not. Also we add listeners to the children of CUSTOMIZED_STATE if there is any without checking whether these states need aggregation or not. We should improve this logic and make sure we only add listeners when it is necessary.
----------------------------------------------------------------
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] zhangmeng916 closed issue #1046: Improve the adding of customized state listener
Posted by GitBox <gi...@apache.org>.
zhangmeng916 closed issue #1046:
URL: https://github.com/apache/helix/issues/1046
----------------------------------------------------------------
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