Riza Suminto has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22956 )

Change subject: IMPALA-14107: Increment table version in fireReloadEvent
......................................................................


Patch Set 9:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22956/9/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
File fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java:

http://gerrit.cloudera.org:8080/#/c/22956/9/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java@7227
PS9, Line 7227: updatedThriftTable
> nit: we are sending an old snapshot if the table is modified in fireReloadE
The reason this is fine is because Coordinator will need to wait for catalog 
topic update version from fireReloadEvent, right?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1bc91023d46a0b57916202d2b64e2cf07b687b74
Gerrit-Change-Number: 22956
Gerrit-PatchSet: 9
Gerrit-Owner: Riza Suminto <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Sai Hemanth Gantasala <[email protected]>
Gerrit-Comment-Date: Wed, 11 Jun 2025 15:01:49 +0000
Gerrit-HasComments: Yes

Reply via email to