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

Change subject: IMPALA-11112: Impala can't resolve json tables created by Hive
......................................................................


Patch Set 1:

(15 comments)

http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py
File tests/metadata/test_hms_integration.py:

http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@676
PS1, Line 676: ,
flake8: E231 missing whitespace after ','


http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@678
PS1, Line 678: w
flake8: E112 expected an indented block


http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@678
PS1, Line 678: h
flake8: E999 IndentationError: expected an indented block


http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@678
PS1, Line 678:
flake8: E501 line too long (102 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@679
PS1, Line 679: e
flake8: E112 expected an indented block


http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@679
PS1, Line 679: =
flake8: E225 missing whitespace around operator


http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@679
PS1, Line 679: /
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@680
PS1, Line 680: /
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@683
PS1, Line 683: w
flake8: E112 expected an indented block


http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@683
PS1, Line 683: ,
flake8: E231 missing whitespace after ','


http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@683
PS1, Line 683: b
flake8: E501 line too long (99 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@684
PS1, Line 684: e
flake8: E112 expected an indented block


http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@684
PS1, Line 684: /
flake8: E226 missing whitespace around arithmetic operator


http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@685
PS1, Line 685:
flake8: E202 whitespace before ')'


http://gerrit.cloudera.org:8080/#/c/18263/1/tests/metadata/test_hms_integration.py@686
PS1, Line 686: /
flake8: E226 missing whitespace around arithmetic operator



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9cac55b58dca88d900db3256ceaa25c17d7864d5
Gerrit-Change-Number: 18263
Gerrit-PatchSet: 1
Gerrit-Owner: Anonymous Coward <pranav.lo...@cloudera.com>
Gerrit-Reviewer: Anonymous Coward <pranav.lo...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Comment-Date: Tue, 22 Feb 2022 08:43:46 +0000
Gerrit-HasComments: Yes

Reply via email to