yashmayya opened a new pull request, #16682: URL: https://github.com/apache/pinot/pull/16682
- https://github.com/apache/pinot/pull/15615 was meant to simply add support for using `LONG` as an alias for `BIGINT` (and other similar type aliases) in the multi-stage query engine (to eliminate [these differences](https://docs.pinot.apache.org/reference/troubleshooting/troubleshoot-multi-stage-query-engine#different-type-names)). - However, an unintentional side effect of the parser changes was that the single-stage engine also now uses `BIGINT` even when `LONG` is specified as the type cast explicitly. - This isn't an issue for fully upgraded clusters because the servers can recognize `BIGINT` as a valid type to cast to, but this could cause query failures during an upgrade where brokers are translating `LONG` to `BIGINT` for SSE queries, but servers don't yet know how to handle`BIGINT` (`IllegalArgumentException: Unable to cast expression to type - BIGINT` in `CastTransformFunction.init`). - This patch fixes the upgrade compatibility issue by introducing a query rewriter for SSE that replaces the Calcite type names with the corresponding Pinot type names. - Note that this query rewriter can simply be removed after the next release if needed. -- 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]
