Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24910 )
Change subject: IMPALA-15387: Reset next_levels_consumed_ for each row group ...................................................................... IMPALA-15387: Reset next_levels_consumed_ for each row group ComplexColumnReader::Reset() did not clear next_levels_consumed_, added by IMPALA-3841. In a new row group the parent collection skipped its nested child in the first NextLevels(), treated itself as empty and read levels again, so a scalar sibling read its levels twice. From the first page boundary on, that sibling returned the next row's value. Testing: - Added a two-level query over customer_nested_multiblock_multipage to parquet-late-materialization-unique-db.test. Expected result computed with pyarrow; a build without the fix returns 11908,9964,29547222652. Change-Id: I7a1fe611da5dfebd99fa62f694a719675822935f Assisted-by: Claude Opus 5 (Claude Code) Reviewed-on: http://gerrit.cloudera.org:8080/24910 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/exec/parquet/parquet-complex-column-reader.h M testdata/workloads/functional-query/queries/QueryTest/parquet-late-materialization-unique-db.test 2 files changed, 16 insertions(+), 0 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/24910 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I7a1fe611da5dfebd99fa62f694a719675822935f Gerrit-Change-Number: 24910 Gerrit-PatchSet: 3 Gerrit-Owner: Aleksandr Efimov <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Xuebin Su <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
