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

Change subject: IMPALA-10976: Sync db/table to latest HMS event for all DDL/DMLs
......................................................................


Patch Set 17:

(5 comments)

http://gerrit.cloudera.org:8080/#/c/20367/17/tests/custom_cluster/test_sync_to_latest_hms_events.py
File tests/custom_cluster/test_sync_to_latest_hms_events.py:

http://gerrit.cloudera.org:8080/#/c/20367/17/tests/custom_cluster/test_sync_to_latest_hms_events.py@582
PS17, Line 582:
flake8: E251 unexpected spaces around keyword / parameter equals


http://gerrit.cloudera.org:8080/#/c/20367/17/tests/custom_cluster/test_sync_to_latest_hms_events.py@582
PS17, Line 582: \
flake8: E502 the backslash is redundant between brackets


http://gerrit.cloudera.org:8080/#/c/20367/17/tests/custom_cluster/test_sync_to_latest_hms_events.py@583
PS17, Line 583: \
flake8: E502 the backslash is redundant between brackets


http://gerrit.cloudera.org:8080/#/c/20367/17/tests/custom_cluster/test_sync_to_latest_hms_events.py@614
PS17, Line 614: s
flake8: E122 continuation line missing indentation or outdented


http://gerrit.cloudera.org:8080/#/c/20367/17/tests/custom_cluster/test_sync_to_latest_hms_events.py@658
PS17, Line 658: ]
flake8: E501 line too long (92 > 90 characters)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia250d0a943838086c187e5cb7c60035e5a564bbf
Gerrit-Change-Number: 20367
Gerrit-PatchSet: 17
Gerrit-Owner: Sai Hemanth Gantasala <saihema...@cloudera.com>
Gerrit-Reviewer: Anonymous Coward <k.venureddy2...@gmail.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Reviewer: Sai Hemanth Gantasala <saihema...@cloudera.com>
Gerrit-Comment-Date: Sun, 17 Dec 2023 19:15:02 +0000
Gerrit-HasComments: Yes

Reply via email to