narutomanchester opened a new pull request, #7591: URL: https://github.com/apache/datafusion-comet/pull/7591
## Which issue does this PR close? Closes #7458. ## Rationale for this change The sequence queries in `one_row_relation.sql` run with the harness's default ConstantFolding exclusion. They exercise the sequence expression rather than the folded `UnsafeArrayData` literal, leaving the literal serializer's `toArray` handling without regression coverage. ## What changes are included in this PR? - Add a folded `sequence(1, 5)` projection to the existing constant-folding-enabled array/map fixture. - Add a nested array of folded sequences to exercise recursive list serialization. Both queries project over the existing three-row Parquet table and use the harness's answer and native-operator checks. - Correct the misleading constant-folding comment in `one_row_relation.sql`. ## How are these changes tested? - `git diff --check` passes. - Local native/JVM builds and the focused `CometSqlFileTestSuite` validation are in progress. - Fork CI: https://github.com/narutomanchester/datafusion-comet/actions/runs/42795273112 Keeping this PR in draft until the regression tests are validated. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
