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

tarmstrong pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git

commit 3c68ddf3aa45021415919bc3250287961ebd1d32
Author: Lars Volker <[email protected]>
AuthorDate: Thu May 30 19:42:44 2019 -0700

    IMPALA-8596: Add debugging output to assertion in test
    
    It looks like the assertion can sporadically fail, but inspecting the
    thrift profile did not show any suspicious fields. This change adds
    debug output to help identify the issue if we hit it again.
    
    Change-Id: If9b649600ad42fea85bfc3020c8d880b690d65dd
    Reviewed-on: http://gerrit.cloudera.org:8080/13482
    Reviewed-by: Impala Public Jenkins <[email protected]>
    Tested-by: Impala Public Jenkins <[email protected]>
---
 tests/query_test/test_observability.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/query_test/test_observability.py 
b/tests/query_test/test_observability.py
index de8a2ab..b26b7b2 100644
--- a/tests/query_test/test_observability.py
+++ b/tests/query_test/test_observability.py
@@ -412,7 +412,7 @@ class TestObservability(ImpalaTestSuite):
         if key in line:
           # Match byte count within parentheses
           m = re.search("\(([0-9]+)\)", line)
-          assert m
+          assert m, "Cannot match pattern for key %s in line '%s'" % (key, 
line)
           # Only keep first (query-level) counter
           if counters[key] == 0:
             counters[key] = int(m.group(1))

Reply via email to