This is an automated email from the ASF dual-hosted git repository.

boroknagyz pushed a commit to branch branch-4.4.0
in repository https://gitbox.apache.org/repos/asf/impala.git

commit 9190a28870c727f9c260cc245eea1c153ee11ff2
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

Reply via email to