Laszlo Gaal has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12892 )

Change subject: Make Docker-based tests available on more hosting OS platforms
......................................................................


Patch Set 3:

(3 comments)

Fixed flake8 style findings.

http://gerrit.cloudera.org:8080/#/c/12892/1/docker/monitor.py
File docker/monitor.py:

http://gerrit.cloudera.org:8080/#/c/12892/1/docker/monitor.py@75
PS1, Line 75:
> flake8: E501 line too long (93 > 90 characters)
Done


http://gerrit.cloudera.org:8080/#/c/12892/1/docker/monitor.py@81
PS1, Line 81:
> flake8: E201 whitespace after '('
Done


http://gerrit.cloudera.org:8080/#/c/12892/1/docker/monitor.py@88
PS1, Line 88:
> flake8: E302 expected 2 blank lines, found 1
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I6d326611772facc193d8e509a827515f10207808
Gerrit-Change-Number: 12892
Gerrit-PatchSet: 3
Gerrit-Owner: Laszlo Gaal <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Laszlo Gaal <[email protected]>
Gerrit-Comment-Date: Mon, 01 Apr 2019 12:07:02 +0000
Gerrit-HasComments: Yes

Reply via email to