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

Change subject: IMPALA-14082: Support batch processing of RELOAD events on same 
table
......................................................................


Patch Set 3:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/23159/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/23159/2//COMMIT_MSG@18
PS2, Line 18: two methods new methods
> nit: "two new methods"
Ack


http://gerrit.cloudera.org:8080/#/c/23159/3/tests/custom_cluster/test_events_custom_configs.py
File tests/custom_cluster/test_events_custom_configs.py:

http://gerrit.cloudera.org:8080/#/c/23159/3/tests/custom_cluster/test_events_custom_configs.py@832
PS3, Line 832:     assert "test" in result.get_data()
> It seems the new test can't catch the bug of https://gerrit.cloudera.org/c/
Fixed it in the latest patchset. Since the name 'test' is present in 
warehouse/dbname, the existing file makes this test pass.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie3e9a99b666a1c928ac2a136bded1e5420f77dab
Gerrit-Change-Number: 23159
Gerrit-PatchSet: 3
Gerrit-Owner: Sai Hemanth Gantasala <saihema...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Sai Hemanth Gantasala <saihema...@cloudera.com>
Gerrit-Comment-Date: Tue, 02 Sep 2025 20:48:58 +0000
Gerrit-HasComments: Yes

Reply via email to