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

Change subject: (Addendum)IMPALA-13500: Fix 
test_invalidate_stale_partition_on_reload() test
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/22198/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/22198/1//COMMIT_MSG@7
PS1, Line 7: (Addendum)IMPALA-13500: Fix 
test_invalidate_stale_partition_on_reload()
           : test
nit: retitle

IMPALA-13500: (addendum) Do not assert CatalogD's partition id


http://gerrit.cloudera.org:8080/#/c/22198/1/tests/custom_cluster/test_events_custom_configs.py
File tests/custom_cluster/test_events_custom_configs.py:

http://gerrit.cloudera.org:8080/#/c/22198/1/tests/custom_cluster/test_events_custom_configs.py@1326
PS1, Line 1326:     log_regex = r"Invalidated objects in cache: \[partition 
%s:p=%%d" \
              :         % test_tbl
I think this is clearer:

    log_regex = "Invalidated objects in cache: \[partition 
{}:p={}".format(test_tbl, '%d')



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I27cdeb2a4bed8afa30a27d05c7399c78af5bcebb
Gerrit-Change-Number: 22198
Gerrit-PatchSet: 1
Gerrit-Owner: Sai Hemanth Gantasala <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Comment-Date: Wed, 15 Jan 2025 20:27:50 +0000
Gerrit-HasComments: Yes

Reply via email to