David Knupp has uploaded a new patch set (#2). Change subject: IMPALA-4684: Wrap checking for HBase nodes ina try/finally block ......................................................................
IMPALA-4684: Wrap checking for HBase nodes ina try/finally block If an exception (other than NoNodeError) was raised while checking for HBase nodes, we weren't cleanly stopping the ZooKeeper client, which in turn created a second exception when the the connection was closed. The second exception masked the original error condition. Tested by forcibly raising unexpected errors while checking for HBase nodes. Change-Id: I46a74d018f9169385a9f10a85718044c31a24dbc --- M testdata/bin/check-hbase-nodes.py 1 file changed, 4 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/47/5547/2 -- To view, visit http://gerrit.cloudera.org:8080/5547 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I46a74d018f9169385a9f10a85718044c31a24dbc Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: David Knupp <dkn...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>