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 0b9efd7906d24be3e0344e81af6009b40bd3b83b
Author: Steve Carlin <[email protected]>
AuthorDate: Thu Feb 19 21:01:10 2026 -0800

    IMPALA-14761: Fixed error in calcite report generator
    
    A recent test framework change caused failure in the calcite
    report generator that is fixed with this patch.
    
    Change-Id: I7be033d30f88e40f86e77b4861ca7000f9b4c138
    Reviewed-on: http://gerrit.cloudera.org:8080/24007
    Reviewed-by: Joe McDonnell <[email protected]>
    Tested-by: Steve Carlin <[email protected]>
---
 bin/calcite_report_generator.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/bin/calcite_report_generator.py b/bin/calcite_report_generator.py
index f370b8bdd..f9db8bdd0 100755
--- a/bin/calcite_report_generator.py
+++ b/bin/calcite_report_generator.py
@@ -119,7 +119,7 @@ def categorize_error_string(error_string):
 
 def process_single_test_node(before_json_contents, after_json_contents, 
out_filename):
   test_node_id = before_json_contents["test_node_id"]
-  test_file, test_class, _, test_function = 
test_node_id.split("[")[0].split("::")
+  test_file, test_class, test_function = test_node_id.split("[")[0].split("::")
   result_category_counts = {}
   for result_category in RESULT_CATEGORIES:
     # Total count, number increased, number decreased
@@ -314,7 +314,7 @@ def main():
 
         # Break apart the test node id to allow aggregating at various levels 
and
         # organizing the directory structure
-        test_file, test_class, _, test_function = 
test_node_id.split("[")[0].split("::")
+        test_file, test_class, test_function = 
test_node_id.split("[")[0].split("::")
 
         # Step 1: Write out individual test html files
         # (When this becomes a diff, we'll have pairs of files to put into 
this)

Reply via email to