bowenliang123 opened a new pull request, #7600: URL: https://github.com/apache/kyuubi/pull/7600
### Why are the changes needed? `scala-collection-compat` is no longer used anywhere in Kyuubi: - **No source code references it**: `grep -rn 'import scala.collection.compat' --include='*.scala'` returns zero hits across the entire repo. - **No transitive dep needs it from us**: Iceberg `1.11.0` (current `iceberg.version` per `pom.xml:167`) no longer ships `scala-collection-compat` in its runtime jar; checked `iceberg-spark-runtime-3.5_2.12-1.11.0.jar` and the `scala/collection/compat` package is absent. - **The 2024 pin is stale**: the original rationale (commit `550f1fce8`, #6677) was that Iceberg 1.6.1 transitively pulled `scala-collection-compat:2.12.0`. Iceberg has since moved on. - **Scala 2.13 path is also unaffected**: `pom.xml:124` `<scala-collection-compat.version>` lives outside any profile, so both `scala-2.12` and `scala-2.13` (lines 2003/2011) shared the same pin. Neither path needs it. 3 hunks across 2 files: - `pom.xml`: drop `<scala-collection-compat.version>` property - `pom.xml`: drop the corresponding `<dependencyManagement>` entry - `extensions/spark/kyuubi-spark-authz/pom.xml`: drop the `<scope>test</scope>` dependency ### How was this patch tested? - [x] `mvn -Pfast clean package -DskipTests -pl extensions/spark/kyuubi-spark-authz -am` — 7 modules build cleanly (kyuubi-parent, kyuubi-util, kyuubi-util-scala, kyuubi-common, kyuubi-events, kyuubi-spark-lineage, kyuubi-spark-authz) - [x] `mvn -pl extensions/spark/kyuubi-spark-authz dependency:tree` confirms `scala-collection-compat_2.12:2.7.0` still lands on the classpath transitively (via Spark SQL `provided` scope) — classpath behaviour unchanged, just let Maven dependency mediation pick the version Spark wants instead of pinning an older 2.12.0 - [ ] full unit-test pass via `mvn test` to follow before merge ### Was this patch authored or co-authored using generative AI tooling? Yes. Assisted-by: OpenCode with MiniMax-M3 -- 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]
