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

Change subject: IMPALA-11728 Set fallback database for functions
......................................................................


Patch Set 5:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/19252/5/tests/query_test/test_udfs.py
File tests/query_test/test_udfs.py:

http://gerrit.cloudera.org:8080/#/c/19252/5/tests/query_test/test_udfs.py@34
PS5, Line 34: from tests.util.filesystem_utils import get_fs_path, IS_S3, 
WAREHOUSE
flake8: F401 'tests.util.filesystem_utils.IS_S3' imported but unused


http://gerrit.cloudera.org:8080/#/c/19252/5/tests/query_test/test_udfs.py@653
PS5, Line 653: E
flake8: E501 line too long (92 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/19252/5/tests/query_test/test_udfs.py@669
PS5, Line 669: U
flake8: E501 line too long (94 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/19252/5/tests/query_test/test_udfs.py@675
PS5, Line 675:
flake8: W292 no newline at end of file



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I37b7e126718fea1c50723cacbaed898b20bb55e5
Gerrit-Change-Number: 19252
Gerrit-PatchSet: 5
Gerrit-Owner: Xiaoqing Gao <[email protected]>
Gerrit-Reviewer: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Xiaoqing Gao <[email protected]>
Gerrit-Comment-Date: Tue, 22 Nov 2022 13:12:42 +0000
Gerrit-HasComments: Yes

Reply via email to