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

Change subject: IMPALA-11013: Support 'MIGRATE TABLE' for external HDFS tables
......................................................................


Patch Set 3:

(6 comments)

http://gerrit.cloudera.org:8080/#/c/20077/3/fe/src/test/java/org/apache/impala/analysis/AnalyzeStmtsTest.java
File fe/src/test/java/org/apache/impala/analysis/AnalyzeStmtsTest.java:

http://gerrit.cloudera.org:8080/#/c/20077/3/fe/src/test/java/org/apache/impala/analysis/AnalyzeStmtsTest.java@5201
PS3, Line 5201:         "CONVERT TO ICEBERG is not supported for 
org.apache.hadoop.mapred.TextInputFormat");
line too long (92 > 90)


http://gerrit.cloudera.org:8080/#/c/20077/3/tests/authorization/test_ranger.py
File tests/authorization/test_ranger.py:

http://gerrit.cloudera.org:8080/#/c/20077/3/tests/authorization/test_ranger.py@1997
PS3, Line 1997:
flake8: E501 line too long (102 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/20077/3/tests/authorization/test_ranger.py@2002
PS3, Line 2002: e
flake8: E501 line too long (93 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/20077/3/tests/authorization/test_ranger.py@2004
PS3, Line 2004: I
flake8: E501 line too long (93 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/20077/3/tests/authorization/test_ranger.py@2015
PS3, Line 2015: f
flake8: E303 too many blank lines (2)


http://gerrit.cloudera.org:8080/#/c/20077/3/tests/authorization/test_ranger.py@2020
PS3, Line 2020: @
flake8: E303 too many blank lines (2)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iacdad996d680fe545cc9a45e6bc64a348a64cd80
Gerrit-Change-Number: 20077
Gerrit-PatchSet: 3
Gerrit-Owner: Gabor Kaszab <[email protected]>
Gerrit-Reviewer: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Fri, 16 Jun 2023 11:44:01 +0000
Gerrit-HasComments: Yes

Reply via email to