AnkitaAdvitot opened a new pull request, #19586: URL: https://github.com/apache/pinot/pull/19586
## Description Fixes #19338. This is a follow-up to #19247 addressing the remaining gaps for SQL array literals and logical types: 1. **`ArrayFunctions.arrayValueConstructor`**: Added handling for `Timestamp` and `UUID` elements so that typed `Timestamp[]` and `UUID[]` arrays are returned instead of falling back to generic `Object[]`. 2. **`LiteralContext`**: Enabled multi-value array literal support for logical types `BOOLEAN` (`boolean[]` and `Boolean[]`), `BIG_DECIMAL` (`BigDecimal[]`), `TIMESTAMP` (`Timestamp[]`), and `UUID` (`UUID[]`). 3. **`LiteralContext.toString()`**: Added string formatting representations for `PRIMITIVE_BOOLEAN_ARRAY`, `BOOLEAN_ARRAY`, `BIG_DECIMAL_ARRAY`, `TIMESTAMP_ARRAY`, and `UUID_ARRAY`. 4. **`FunctionUtils`**: Registered `UUID[].class` mapping to `ColumnDataType.UUID_ARRAY` in `COLUMN_DATA_TYPE_MAP`. ## Validation - Ran unit tests in `pinot-common`: `LiteralContextTest`, `ArrayFunctionsTest`, `FunctionUtilsTest`, `LiteralSerDeTest` (all 48 passed). - Verified Checkstyle with 0 violations (`mvn checkstyle:check`). -- 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]
