MonkeyCanCode commented on PR #4746:
URL: https://github.com/apache/polaris/pull/4746#issuecomment-4738036197
@adutra
so on both spark 3 and spark 4, we have following to exclude ANTLR from
quarks bom into those spark integration test:
```
implementation(enforcedPlatform(libs.quarkus.bom)) {
exclude(group = "org.antlr", module = "antlr4-runtime")
exclude(group = "org.scala-lang", module = "scala-library")
exclude(group = "org.scala-lang", module = "scala-reflect")
}
```
The two ANTLR entries are needed for their respective Spark test modules
which we can't combine them into one as they are not compatible. Same had been
discovered by @adam-christian-software in the past via
https://github.com/apache/polaris/pull/3188#discussion_r2593927476
Any other concern before we merge this one?
--
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]
