Till Westmann has posted comments on this change. Change subject: This change fixes the sporadic connection refused errors after managix starts up an Asterix instance. This can mostly be seen during asterix-installer tests. The fix basically lets managix wait and listens through zookeeper for a signal from the cluster c ......................................................................
Patch Set 1: Code-Review+1 Ok, I understand better now. Thanks! I agree that there in no intention of removing Zookeeper for Managix. However our other approach to starting the cluster node via YARN doesn't use Zookeeper (yet?). Also, I think that we might want to have another way to start an stop cluster nodes in the future that also might not use Zookeeper. So I think that introducing a dependency on Zookeeper is a little too much - especially given that we use Zookeeper in a very simple use case. However, I think that this change clearly solves a problem for now and we can re-discuss the mechanism when it becomes necessary. -- To view, visit https://asterix-gerrit.ics.uci.edu/365 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0975308e1dc2650d8c2e50d58ff2c696b22eaaa4 Gerrit-PatchSet: 1 Gerrit-Project: hyracks Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi <[email protected]> Gerrit-Reviewer: Chris Hillery <[email protected]> Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]> Gerrit-HasComments: No
