This is an automated email from the ASF dual-hosted git repository. joemcdonnell pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/impala.git
commit 06bbbea257547774aa8e2bad8820e38d5bbf5a51 Author: Kurt Deschler <[email protected]> AuthorDate: Tue Apr 16 12:30:40 2024 -0500 IMPALA-12679: Improve test_rows_sent_counters assert This patch changes the assert for failed test test_rows_sent_counters so that the actual RPC count is displayed in the assert output. The root cause of the failure will be addressed once sufficient data is collected with the new output. Testing: Ran test_rows_sent_counters with modified expected RPC count range to simulate failure. Change-Id: Ic6b48cf4039028e749c914ee60b88f04833a0069 Reviewed-on: http://gerrit.cloudera.org:8080/21310 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- tests/query_test/test_fetch.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/query_test/test_fetch.py b/tests/query_test/test_fetch.py index 22f2b3b38..c70e5158f 100644 --- a/tests/query_test/test_fetch.py +++ b/tests/query_test/test_fetch.py @@ -66,7 +66,8 @@ class TestFetch(ImpalaTestSuite): materialization_timer = re.search("RowMaterializationTimer: (.*)", runtime_profile) assert materialization_timer and len(materialization_timer.groups()) == 1 and \ parse_duration_string_ms(materialization_timer.group(1)) > 1000 - assert re.search("RPCCount: [5-9]", runtime_profile) + rpc_count = int(re.search("RPCCount: ([0-9]+)", runtime_profile).group(1)) + assert rpc_count >= 5 and rpc_count <= 9 rpc_read_timer = re.search("RPCReadTimer: (.*)", runtime_profile) assert rpc_read_timer and len(rpc_read_timer.groups()) == 1
