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

Change subject: IMPALA-11034: Resolve schema of old data files in migrated 
Iceberg tables
......................................................................


Patch Set 8:

(10 comments)

http://gerrit.cloudera.org:8080/#/c/18639/8/tests/common/file_utils.py
File tests/common/file_utils.py:

http://gerrit.cloudera.org:8080/#/c/18639/8/tests/common/file_utils.py@29
PS8, Line 29: o
flake8: E501 line too long (97 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/18639/8/tests/common/file_utils.py@33
PS8, Line 33: ,
flake8: E231 missing whitespace after ','


http://gerrit.cloudera.org:8080/#/c/18639/8/tests/common/file_utils.py@33
PS8, Line 33: a
flake8: E501 line too long (105 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/18639/8/tests/common/file_utils.py@49
PS8, Line 49: r
flake8: E501 line too long (175 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/18639/8/tests/common/file_utils.py@53
PS8, Line 53: '
flake8: E501 line too long (126 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/18639/8/tests/query_test/test_iceberg.py
File tests/query_test/test_iceberg.py:

http://gerrit.cloudera.org:8080/#/c/18639/8/tests/query_test/test_iceberg.py@107
PS8, Line 107: t
flake8: E501 line too long (111 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/18639/8/tests/query_test/test_iceberg.py@108
PS8, Line 108: m
flake8: E501 line too long (113 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/18639/8/tests/query_test/test_iceberg.py@109
PS8, Line 109: t
flake8: E501 line too long (111 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/18639/8/tests/query_test/test_iceberg.py@110
PS8, Line 110: m
flake8: E501 line too long (113 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/18639/8/tests/query_test/test_iceberg.py@111
PS8, Line 111: q
flake8: E501 line too long (103 > 90 characters)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I77570bbfc2fcc60c2756812d7210110e8cc11ccc
Gerrit-Change-Number: 18639
Gerrit-PatchSet: 8
Gerrit-Owner: Gergely Fürnstáhl <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Wed, 29 Jun 2022 16:41:25 +0000
Gerrit-HasComments: Yes

Reply via email to