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

Change subject: IMPALA-11429: Set table owner after creating an Iceberg table
......................................................................


Patch Set 3:

(13 comments)

http://gerrit.cloudera.org:8080/#/c/18837/3/tests/query_test/test_iceberg.py
File tests/query_test/test_iceberg.py:

http://gerrit.cloudera.org:8080/#/c/18837/3/tests/query_test/test_iceberg.py@43
PS3, Line 43: from tests.shell.util import run_impala_shell_cmd
flake8: E402 module level import not at top of file


http://gerrit.cloudera.org:8080/#/c/18837/3/tests/query_test/test_iceberg.py@45
PS3, Line 45: class TestIcebergTable(ImpalaTestSuite):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/18837/3/tests/query_test/test_iceberg.py@742
PS3, Line 742: <<<<<<< 0ce2403b58afd94877e36e1647ec34e8a9df7dba
flake8: E305 expected 2 blank lines after class or function definition, found 1


http://gerrit.cloudera.org:8080/#/c/18837/3/tests/query_test/test_iceberg.py@742
PS3, Line 742: <
flake8: E999 SyntaxError: invalid syntax


http://gerrit.cloudera.org:8080/#/c/18837/3/tests/query_test/test_iceberg.py@742
PS3, Line 742: <
flake8: E227 missing whitespace around bitwise or shift operator


http://gerrit.cloudera.org:8080/#/c/18837/3/tests/query_test/test_iceberg.py@742
PS3, Line 742: <
flake8: E225 missing whitespace around operator


http://gerrit.cloudera.org:8080/#/c/18837/3/tests/query_test/test_iceberg.py@750
PS3, Line 750: =
flake8: E225 missing whitespace around operator


http://gerrit.cloudera.org:8080/#/c/18837/3/tests/query_test/test_iceberg.py@750
PS3, Line 750: =
flake8: E225 missing whitespace around operator


http://gerrit.cloudera.org:8080/#/c/18837/3/tests/query_test/test_iceberg.py@750
PS3, Line 750: =
flake8: E225 missing whitespace around operator


http://gerrit.cloudera.org:8080/#/c/18837/3/tests/query_test/test_iceberg.py@751
PS3, Line 751: d
flake8: E113 unexpected indentation


http://gerrit.cloudera.org:8080/#/c/18837/3/tests/query_test/test_iceberg.py@759
PS3, Line 759: d
flake8: E126 continuation line over-indented for hanging indent


http://gerrit.cloudera.org:8080/#/c/18837/3/tests/query_test/test_iceberg.py@780
PS3, Line 780: >
flake8: E225 missing whitespace around operator


http://gerrit.cloudera.org:8080/#/c/18837/3/tests/query_test/test_iceberg.py@780
PS3, Line 780: -
flake8: E226 missing whitespace around arithmetic operator



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5cac198a4a53be3599cb582864ee5f8c269202c0
Gerrit-Change-Number: 18837
Gerrit-PatchSet: 3
Gerrit-Owner: Gabor Kaszab <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Thu, 11 Aug 2022 16:00:34 +0000
Gerrit-HasComments: Yes

Reply via email to