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

Change subject: IMPALA-10953 Fix impalad crashes due to TmpFileMgr 
initialization fails
......................................................................


Patch Set 1:

(6 comments)

http://gerrit.cloudera.org:8080/#/c/17906/1/tests/custom_cluster/test_scratch_disk.py
File tests/custom_cluster/test_scratch_disk.py:

http://gerrit.cloudera.org:8080/#/c/17906/1/tests/custom_cluster/test_scratch_disk.py@456
PS1, Line 456: ,
flake8: E231 missing whitespace after ','


http://gerrit.cloudera.org:8080/#/c/17906/1/tests/custom_cluster/test_scratch_disk.py@456
PS1, Line 456:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/17906/1/tests/custom_cluster/test_scratch_disk.py@456
PS1, Line 456:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/17906/1/tests/custom_cluster/test_scratch_disk.py@466
PS1, Line 466:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/17906/1/tests/custom_cluster/test_scratch_disk.py@474
PS1, Line 474: r
flake8: F841 local variable 'result' is assigned to but never used


http://gerrit.cloudera.org:8080/#/c/17906/1/tests/custom_cluster/test_scratch_disk.py@476
PS1, Line 476:
flake8: W391 blank line at end of file



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7852bf4206a603e6e2652ce9232077777a06ce67
Gerrit-Change-Number: 17906
Gerrit-PatchSet: 1
Gerrit-Owner: Yida Wu <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Tue, 05 Oct 2021 23:25:52 +0000
Gerrit-HasComments: Yes

Reply via email to