albertobastos opened a new pull request, #15692: URL: https://github.com/apache/pinot/pull/15692
A few weeks ago case-insensitive support for MSE [was added](https://github.com/apache/pinot/pull/14830) by using Calcite built-in support for that. However, as [pointed out](https://github.com/apache/pinot/pull/14830#issuecomment-2788228861) by @ankitsultana, it introduced some backwards incompatibility due to some changes required by how Calcite handles case-insensitiveness (returning from then on always lower-case identifiers). This PR undoes the previous one and applies another strategy, bypassing Calcite built-in support and instead just doing case-insensitive lookups in very specific places where the database catalog is used. Only two visible changes happen with this strategy: - The list of tables returned by the Pinot Server's `/tables` endpoint will now be all lower-case. That may seem acceptable given that the cluster itself has been launched flagged as case insensitive. - Queries using MSE and refering to non-existent tables will return, instead of a TableDoesNotExistError, a QueryValidationError with the TableDoesNotExistError wrapped inside it. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
