impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David Knupp (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-2013: Reintroduce steps for checking HBase health in
Date Tue, 13 Sep 2016 23:07:55 GMT
David Knupp has posted comments on this change.

Change subject: IMPALA-2013: Reintroduce steps for checking HBase health in

Patch Set 8:

(1 comment)
File testdata/bin/

Line 88:         LOGGER.debug("Success: " + str(zk_client))
> Otherwise it looks like we're trying to connect but never succ
That's a good catch, but I think it might be OK as-is. This is the output of a typical run:

    Contents of HDFS root: [u'hbase', u'home', u'test-warehouse', u'tmp', u'user']
    Connecting to Zookeeper host(s).
    Waiting for HBase node: /hbase/master
    Success: /hbase/master
    Waiting for HBase node: /hbase/rs
    Success: /hbase/rs 

The missing line would have been:

    Success: <kazoo.client.KazooClient object at 0x7f12dda65250>

If the connection attempt didn't succeed, I don't think it would be a mystery -- you'd get
the exception msg, and the script would exit with an error. Is that acceptable? (Honestly,
I think might have intended to take that line out.)

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I9b81f3cfb6ea0ba7b18ce5fcd5d268f515c8b0c3
Gerrit-PatchSet: 8
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: David Knupp <>
Gerrit-Reviewer: Alex Behm <>
Gerrit-Reviewer: David Knupp <>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Ishaan Joshi <>
Gerrit-Reviewer: Michael Brown <>
Gerrit-HasComments: Yes

View raw message