adutra commented on PR #4746: URL: https://github.com/apache/polaris/pull/4746#issuecomment-4706326682
FYI we had problems before between the ANTLR version used by Quarkus and the one used by Spark. That was the main reason why we introduced the `polaris-runtime-spark-tests` module. Pinning the version to the one from Delta for the whole project may cause trouble later on with Quarkus, I fear. Also, we have 2 versions of ANTLR in `libs.version.toml`: https://github.com/apache/polaris/blob/6cfb0bd561aced0eda90a5bcc6926a84d3006989/gradle/libs.versions.toml#L45-L46 Shouldn't we clean that up? -- 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]
