Tim Armstrong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12078 )

Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/12078/1/bin/start-impala-cluster.py
File bin/start-impala-cluster.py:

http://gerrit.cloudera.org:8080/#/c/12078/1/bin/start-impala-cluster.py@24
PS1, Line 24: import json
> flake8: F401 'json' imported but unused
Done


http://gerrit.cloudera.org:8080/#/c/12078/1/bin/start-impala-cluster.py@375
PS1, Line 375: def 
wait_for_cluster(timeout_in_seconds=CLUSTER_WAIT_TIMEOUT_IN_SECONDS):
> flake8: E302 expected 2 blank lines, found 1
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia28a7b75f2e2804834e5df57ef7998b360112e5c
Gerrit-Change-Number: 12078
Gerrit-PatchSet: 1
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Lars Volker <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Thu, 13 Dec 2018 01:47:17 +0000
Gerrit-HasComments: Yes

Reply via email to