Can we get this change merged? I wasted 2 days trying to debug a test case failure that happens on the server and doesn’t happen on my machine.
Turns out because I added a couple of logging messages for datasets memory allocation and deallocation, the NC was taking more time to register with the CC and the query is submitted before the cluster is active. I couldn’t figure out the reason right away because the returned error message is not formatted in JSON, which triggered another bug that raises a JSON format exception instead of the actual error message. Regard, Murtadha > On Sep 14, 2015, at 3:36 PM, Till Westmann (Code Review) > <[email protected]> wrote: > > 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 3: Code-Review+2 > > -- > 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: 3 > 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
