Aleksandr Efimov has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24564 )

Change subject: IMPALA-15156: Fix CTE size metadata
......................................................................


Patch Set 6:

> Patch Set 6: Verified-1
>
> Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/13821/

I had a look at the failed build. Both downstream jobs fail only test_cte_plans 
and test_cte_distributed.

This seems to come from the rebase rather than the changes in 24564. 
IMPALA-2210 changed the default file format from TEXT to PARQUET, while both 
tests create emps without an explicit STORED AS. The resulting file is now 
1.13KB instead of 58B, which also changes the cardinality and resource 
estimates in the expected plans.

The smallest fix seems to be adding STORED AS TEXTFILE to those two CREATE 
TABLE statements in 22094. That would keep the tests independent of the global 
default and keep 24564 focused.


--
To view, visit http://gerrit.cloudera.org:8080/24564
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ide226449f7b7a33fffd67f403b03fd5b2fee53ee
Gerrit-Change-Number: 24564
Gerrit-PatchSet: 6
Gerrit-Owner: Aleksandr Efimov <[email protected]>
Gerrit-Reviewer: Aleksandr Efimov <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Steve Carlin <[email protected]>
Gerrit-Comment-Date: Tue, 04 Aug 2026 03:59:58 +0000
Gerrit-HasComments: No

Reply via email to