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 24:

(12 comments)

http://gerrit.cloudera.org:8080/#/c/20367/24/tests/metadata/test_ddl.py
File tests/metadata/test_ddl.py:

http://gerrit.cloudera.org:8080/#/c/20367/24/tests/metadata/test_ddl.py@153
PS24, Line 153: t
flake8: F821 undefined name 'test_truncate_cleans_hdfs_files'


http://gerrit.cloudera.org:8080/#/c/20367/24/tests/metadata/test_ddl.py@156
PS24, Line 156: @
flake8: F811 redefinition of unused 'test_truncate_cleans_hdfs_files' from line 
150


http://gerrit.cloudera.org:8080/#/c/20367/24/tests/metadata/test_ddl.py@252
PS24, Line 252: @
flake8: F811 redefinition of unused 'test_metadata_after_alter_database' from 
line 249


http://gerrit.cloudera.org:8080/#/c/20367/24/tests/metadata/test_ddl.py@270
PS24, Line 270: @
flake8: F811 redefinition of unused 'test_alter_table_set_owner' from line 267


http://gerrit.cloudera.org:8080/#/c/20367/24/tests/metadata/test_ddl.py@286
PS24, Line 286: @
flake8: F811 redefinition of unused 'test_alter_view_set_owner' from line 283


http://gerrit.cloudera.org:8080/#/c/20367/24/tests/metadata/test_ddl.py@498
PS24, Line 498: @
flake8: F811 redefinition of unused 'test_drop_partition_with_purge' from line 
493


http://gerrit.cloudera.org:8080/#/c/20367/24/tests/metadata/test_ddl.py@646
PS24, Line 646: r
flake8: E501 line too long (92 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/20367/24/tests/metadata/test_ddl.py@651
PS24, Line 651: @
flake8: F811 redefinition of unused 'test_create_alter_bulk_partition' from 
line 644


http://gerrit.cloudera.org:8080/#/c/20367/24/tests/metadata/test_ddl.py@698
PS24, Line 698: @
flake8: F811 redefinition of unused 'test_alter_table_set_fileformat' from line 
694


http://gerrit.cloudera.org:8080/#/c/20367/24/tests/metadata/test_ddl.py@724
PS24, Line 724: @
flake8: F811 redefinition of unused 'test_alter_table_create_many_partitions' 
from line 721


http://gerrit.cloudera.org:8080/#/c/20367/24/tests/metadata/test_ddl.py@748
PS24, Line 748: @
flake8: F811 redefinition of unused 'test_create_alter_tbl_properties' from 
line 745


http://gerrit.cloudera.org:8080/#/c/20367/24/tests/metadata/test_ddl.py@808
PS24, Line 808: @
flake8: F811 redefinition of unused 'test_alter_tbl_properties_reload' from 
line 805



--
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: 24
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: Fri, 05 Jan 2024 22:20:12 +0000
Gerrit-HasComments: Yes

Reply via email to