xiangfu0 opened a new pull request, #19246: URL: https://github.com/apache/pinot/pull/19246
## Summary - render `STRING_ARRAY` values in `LiteralContext` - add unit coverage for Thrift `STRING_ARRAY_VALUE` conversion and rendering - add an `ArrayTest` integration test that validates ingested multi-value STRING data and composed string-array literals in projections and `ARRAYS_OVERLAP` filters with both query engines ## Reproduction Before this change, a table-backed single-stage query such as: ```sql SELECT intCol, stringArrayCol, ARRAY['query-a', 'query-b'] FROM ArrayTest WHERE intCol = 0 ``` folds the array constructor into a Thrift `STRING_ARRAY_VALUE`. When the server builds the selection result schema, `LiteralContext.toString()` throws: ```text IllegalStateException: Unsupported PinotDataType: STRING_ARRAY ``` Literal-only queries avoid the server selection path, which is why the failure requires a table-backed projection. ## Validation - `LiteralContextTest`: 11 tests passed - `./mvnw -pl pinot-integration-tests -am -Dtest=ArrayTest#testFoldedStringArrayLiteralWithIngestedColumn -Dsurefire.failIfNoSpecifiedTests=false test`: 2 tests passed, covering both query engines - `./mvnw spotless:apply -pl pinot-common,pinot-integration-tests` - `./mvnw checkstyle:check -pl pinot-common,pinot-integration-tests` - `./mvnw license:format -pl pinot-common,pinot-integration-tests` - `./mvnw license:check -pl pinot-common,pinot-integration-tests` -- 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]
