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


The following commit(s) were added to refs/heads/master by this push:
     new fe3f82179 IMPALA-13539: Fix TestCalcitePlanner.test_calcite_frontend 
for non-HDFS
fe3f82179 is described below

commit fe3f82179fa0726af0645d01f90a2bf966175b75
Author: Joe McDonnell <[email protected]>
AuthorDate: Fri Nov 8 08:11:54 2024 -0800

    IMPALA-13539: Fix TestCalcitePlanner.test_calcite_frontend for non-HDFS
    
    TestCalcitePlanner.test_calcite_frontend runs an explain plan
    and matches the expected output. On HDFS that includes "SCAN HDFS",
    but on S3 and other non-HDFS filesystems, that can be "SCAN S3".
    Similarly, "HDFS partitions" can be "S3 partitions". This changes
    the expected output to make it insensitive to these differences.
    
    Testing:
     - Ran test case locally and on S3
    
    Change-Id: I7ddf919558a7ad622d7571283ad8a2e069c69f14
    Reviewed-on: http://gerrit.cloudera.org:8080/22044
    Reviewed-by: Impala Public Jenkins <[email protected]>
    Tested-by: Impala Public Jenkins <[email protected]>
---
 testdata/workloads/functional-query/queries/QueryTest/calcite.test | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/testdata/workloads/functional-query/queries/QueryTest/calcite.test 
b/testdata/workloads/functional-query/queries/QueryTest/calcite.test
index 207af6d07..0279778e1 100644
--- a/testdata/workloads/functional-query/queries/QueryTest/calcite.test
+++ b/testdata/workloads/functional-query/queries/QueryTest/calcite.test
@@ -927,7 +927,7 @@ row_regex: .*PlannerType: CalcitePlanner.*
 explain select * from functional.alltypestiny;
 ---- RESULTS: VERIFY_IS_SUBSET
 row_regex:.*01:EXCHANGE.*
-row_regex:.*00:SCAN HDFS.*
+row_regex:.*00:SCAN.*
 ---- RUNTIME_PROFILE
 row_regex: .*PlannerType: CalcitePlanner.*
 ====
@@ -936,8 +936,8 @@ set explain_level=3;
 explain select * from functional.alltypestiny;
 ---- RESULTS: VERIFY_IS_SUBSET
 row_regex:.*01:EXCHANGE.*
-row_regex:.*00:SCAN HDFS.*
-row_regex:.*HDFS partitions=4/4.*
+row_regex:.*00:SCAN.*
+row_regex:.*partitions=4/4.*
 ---- RUNTIME_PROFILE
 row_regex: .*PlannerType: CalcitePlanner.*
 ====

Reply via email to