Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11085 )

Change subject: WIP: IMPALA-7317: comment on long lines and whitespace
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11085/1/bin/jenkins/critique-gerrit-review.py
File bin/jenkins/critique-gerrit-review.py:

http://gerrit.cloudera.org:8080/#/c/11085/1/bin/jenkins/critique-gerrit-review.py@127
PS1, Line 127:     raise Exception("Did not expect flake8-diff to write to 
stderr:\n{0}".format(stderr))
flake8: E305 expected 2 blank lines after class or function definition, found 1



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I36bb99560ab09d7753ff93227d1c4972582770f2
Gerrit-Change-Number: 11085
Gerrit-PatchSet: 1
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Mon, 30 Jul 2018 23:59:03 +0000
Gerrit-HasComments: Yes

Reply via email to