Tim Armstrong has posted comments on this change.

Change subject: IMPALA-3839: Fix race condition in impala_cluster.py
......................................................................


Patch Set 1: Code-Review+2

(2 comments)

http://gerrit.cloudera.org:8080/#/c/3592/1/tests/common/impala_cluster.py
File tests/common/impala_cluster.py:

Line 99:         try:
It would be kind of nice to share this with start-impala-cluster.py but I don't 
know if there's a logical place to share python code between these two places.


PS1, Line 110: >=1
missing space


-- 
To view, visit http://gerrit.cloudera.org:8080/3592
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3c0d5e43f8c58b4922ab5db78236915fcc8b588d
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Lars Volker <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: Yes

Reply via email to