Quanlong Huang 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 39:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/20367/38/fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java
File fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java:

http://gerrit.cloudera.org:8080/#/c/20367/38/fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java@1792
PS38, Line 1792:               "So table schema reload can be skipped. SD 
before: {}, SD after: {}",
> Unfortunately, few tests I introduced in this patch are dependent on this l
What about submitting a patch for IMPALA-12487 and merged it before this? It 
might be safer and easier to backport IMPALA-12487 with an individual patch 
(with tests).



--
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: 39
Gerrit-Owner: Sai Hemanth Gantasala <[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: Sai Hemanth Gantasala <[email protected]>
Gerrit-Comment-Date: Tue, 06 Feb 2024 04:12:30 +0000
Gerrit-HasComments: Yes

Reply via email to