I would like to do so but been waiting for the release to be done. This is a two part change (Hyracks and Asterix).
Should I go ahead or wait? Amoudi, Abdullah. On Mon, Sep 28, 2015 at 6:50 AM, Murtadha Hubail <[email protected]> wrote: > 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 > >
