somandal commented on PR #16662: URL: https://github.com/apache/pinot/pull/16662#issuecomment-3225504923
> @somandal > > > Also, just FYI, for broker there exists a class ClusterChangeMediator which handles this. It's a little different as it handles a bunch of other types of ZNode updates as well > > on that note, do we want to standardize to this class instead? That way, we can make it consistent. lmk I felt like this would be overkill as for now I don't see a use case for have all kinds of listeners for Server and Controller. If the need arises, perhaps we can do the refactoring then? Feel free to take a look though and see if you'd prefer doing that -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
