MonkeyCanCode commented on PR #4746: URL: https://github.com/apache/polaris/pull/4746#issuecomment-4708392272
> 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? Two versions are expected as delta 3.5 and 4.x are using different one. I don't recall we can change them as they are used by delta (from what I recalled while working on spark4, I was not able to just use the latest one). I can check later this week to get the exact issue that I saw earlier. -- 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]
