Jackie-Jiang opened a new pull request, #19333:
URL: https://github.com/apache/pinot/pull/19333

   ## Summary
   
   Add `JSON` and `UUID` data type support to the `ARRAY_AGG` aggregation 
function, for both the regular and `DISTINCT` variants:
   - `JSON` is dispatched to the STRING variants. Its stored type is `STRING` 
and there is no `JSON_ARRAY` column data type, so the result surfaces as 
`STRING_ARRAY`.
   - `UUID` is dispatched to the BYTES variants with the logical type passed 
through (mirroring how `BOOLEAN`/`TIMESTAMP` are passed to the INT/LONG 
variants), so the final result column type resolves to `UUID_ARRAY` and values 
render as canonical UUID strings instead of hex.
   
   The BYTES `ARRAY_AGG` functions now take the `DataType` in their 
constructors instead of hardcoding `BYTES`. No changes are needed at the 
MSE/DataTable boundaries: they are keyed on stored types (`UUID_ARRAY` -> 
`BYTES_ARRAY`), which already handle the `ObjectArrayList<ByteArray>` result 
shape.
   
   Also extends `ArrayAggQueriesTest` with end-to-end coverage for `JSON`, 
`BYTES` and `UUID`.
   


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