yaooqinn opened a new pull request, #11526:
URL: https://github.com/apache/incubator-gluten/pull/11526

   ## What changes are proposed in this pull request?
   
   Add comprehensive test coverage for `collect_list` aggregate function in 
`VeloxAggregateFunctionsSuite`.
   
   ### New Tests:
   1. **collect_list with different primitive types** - Tests Boolean, Byte, 
Short, Int, Long, Float, Double, Decimal, String, Date, Timestamp, Binary types 
and null handling
   2. **collect_list with nested complex types** - Tests Array, Struct, and Map 
types
   3. **collect_list with large result sets** - Tests performance with 1000 
elements
   4. **collect_list global aggregation without group by** - Tests global 
aggregation and empty table scenarios
   5. **collect_list fallback for unsupported types** - Tests TimestampNTZ with 
explicit fallback verification
   
   ## How was this patch tested?
   
   - Code compiles successfully with spotless formatting applied
   - Tests follow existing patterns in VeloxAggregateFunctionsSuite
   - Fallback test explicitly verifies no HashAggregateExecTransformer is 
present for unsupported types
   
   ## Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: GitHub Copilot (Claude)


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