Sai Hemanth Gantasala has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21175 )

Change subject: IMPALA-12829: Skip processing transaction events if the table 
is HMS sync disabled.
......................................................................


Patch Set 27:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/21175/26/tests/custom_cluster/test_events_custom_configs.py
File tests/custom_cluster/test_events_custom_configs.py:

http://gerrit.cloudera.org:8080/#/c/21175/26/tests/custom_cluster/test_events_custom_configs.py@1708
PS26, Line 1708:     # HMS Event Processor.
> nit: don't need this change
Ack


http://gerrit.cloudera.org:8080/#/c/21175/26/tests/custom_cluster/test_events_custom_configs.py@1716
PS26, Line 1716: 1")
> Please revert this as well.
Ack


http://gerrit.cloudera.org:8080/#/c/21175/26/tests/custom_cluster/test_events_custom_configs.py@1725
PS26, Line 1725: 0
> Please revert this
Ack



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5d0ecb3b756755bc04c66a538a9ae6b88011a019
Gerrit-Change-Number: 21175
Gerrit-PatchSet: 27
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: Riza Suminto <[email protected]>
Gerrit-Reviewer: Sai Hemanth Gantasala <[email protected]>
Gerrit-Comment-Date: Thu, 21 Aug 2025 17:01:10 +0000
Gerrit-HasComments: Yes

Reply via email to