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

Change subject: IMPALA-9867: Add Support for Spilling to S3: Milestone 1
......................................................................


Patch Set 12:

(8 comments)

http://gerrit.cloudera.org:8080/#/c/16318/12/tests/custom_cluster/test_scratch_disk.py
File tests/custom_cluster/test_scratch_disk.py:

http://gerrit.cloudera.org:8080/#/c/16318/12/tests/custom_cluster/test_scratch_disk.py@263
PS12, Line 263: #
flake8: E265 block comment should start with '# '


http://gerrit.cloudera.org:8080/#/c/16318/12/tests/custom_cluster/test_scratch_disk.py@270
PS12, Line 270: -
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/16318/12/tests/custom_cluster/test_scratch_disk.py@299
PS12, Line 299: -
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/16318/12/tests/custom_cluster/test_scratch_disk.py@330
PS12, Line 330: -
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/16318/12/tests/custom_cluster/test_scratch_disk.py@354
PS12, Line 354: m
flake8: E501 line too long (113 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/16318/12/tests/custom_cluster/test_scratch_disk.py@355
PS12, Line 355: r
flake8: E501 line too long (111 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/16318/12/tests/custom_cluster/test_scratch_disk.py@358
PS12, Line 358: -
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/16318/12/tests/custom_cluster/test_scratch_disk.py@366
PS12, Line 366: h
flake8: E501 line too long (109 > 90 characters)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I419b1d5dbbfe35334d9f964c4b65e553579fdc89
Gerrit-Change-Number: 16318
Gerrit-PatchSet: 12
Gerrit-Owner: Yida Wu <[email protected]>
Gerrit-Reviewer: Abhishek Rawat <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Yida Wu <[email protected]>
Gerrit-Comment-Date: Mon, 21 Sep 2020 20:26:45 +0000
Gerrit-HasComments: Yes

Reply via email to