cloud-fan opened a new pull request, #56534:
URL: https://github.com/apache/spark/pull/56534
### What changes were proposed in this pull request?
SPARK-55706 disabled the DB2 JDBC driver tests because `com.ibm.db2:jcc`
bundles
its own unshaded copy of lz4-java (`net.jpountz.*`), which collides with the
lz4-java Spark ships on the shared test classpath and breaks
`LZ4BlockInputStream.newBuilder()`.
This re-enables those tests by repackaging the driver with `net.jpountz`
relocated so the two copies can coexist:
- New module `connector/db2-jcc-shaded` relocates `net.jpountz` ->
`org.sparkproject.db2jcc.net.jpountz` via maven-shade-plugin (and the
matching
sbt-assembly shade rules in `SparkBuild.scala`). It is test-only and never
published (`maven.deploy.skip=true`).
- `sql/core` and `connector/docker-integration-tests` depend on the shaded
driver instead of `com.ibm.db2:jcc` directly.
- Reverts the test-disabling changes from SPARK-55706.
### Why are the changes needed?
To restore DB2 JDBC test coverage disabled in SPARK-55706, without the
lz4-java
classpath conflict.
### Does this PR introduce _any_ user-facing change?
No. Test-only; the shaded driver module is never published.
### How was this patch tested?
The DB2 integration/connection-provider suites disabled by SPARK-55706 are
re-enabled. **WIP** — opened to let CI validate the Maven + SBT build wiring
(the shaded-test-dependency mechanism could not be built locally).
This pull request and its description were written by Isaac.
--
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]
