Michael Smith 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: > > > 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. I'll look into it, thanks for digging. -- 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 16:03:43 +0000 Gerrit-HasComments: No
