jackjlli commented on a change in pull request #4422: In ClusterChangeMediator,
stop enqueue/process changes if already stopped
URL: https://github.com/apache/incubator-pinot/pull/4422#discussion_r302335684
##########
File path:
pinot-broker/src/main/java/org/apache/pinot/broker/broker/helix/ClusterChangeMediator.java
##########
@@ -145,15 +145,15 @@ private void processClusterChange(ChangeType changeType,
List<ClusterChangeHandl
/**
* Starts the cluster change mediator.
*/
- public void start() {
+ public synchronized void start() {
Review comment:
Do we need synchronized block here? These start and stop methods would only
be called in brokerStarter though. Or maybe just adding the sync block on
`_stopped` would be enough?
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
