narutomanchester opened a new pull request, #7592:
URL: https://github.com/apache/datafusion-comet/pull/7592

   ## 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.
   - JVM test compilation passes on Spark 4.1 / JDK 17.
   - Native build, formatting, and focused regression validation are in 
progress.
   - Fork CI: 
https://github.com/narutomanchester/datafusion-comet/actions/runs/42797176634
   
   Keeping this PR in draft until the regression tests are validated.
   
   Replaces #7591 after renaming the source branch; the patch is unchanged.
   


-- 
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]

Reply via email to