gortiz opened a new pull request, #11417: URL: https://github.com/apache/pinot/pull/11417
While V1 is case insensitive, V2 is case sensitive. I'm not sure if we consider this a bug or a feature. This PR assumes it is a bug and therefore changes the V2 behavior to be case insensitive. I've also modified `OfflineClusterIntegrationTest` to run the case insensitivity tests with V2. Given that these tests were testing the case insensitivity but also the fact that V1 ignores the schema, I had to split them: * testCaseInsensitivity and testCaseInsensitivityWithColumnNameContainsTableName test case sensibility * testSchemaIgnoredInV1 and testIgnoresSchemaInColumnNameWithTestNameInV1 test that schema is ignored in V1 In case we consider case sensibility in V2 a feature, feel free to close this PR -- 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]
