kevinjqliu commented on code in PR #16214:
URL: https://github.com/apache/iceberg/pull/16214#discussion_r4530486945
##########
.github/workflows/publish-iceberg-rest-fixture-docker.yml:
##########
@@ -49,8 +49,13 @@ jobs:
distribution: zulu
java-version: 21
- uses:
gradle/actions/setup-gradle@0723195856401067f7a2779048b490ace7a47d7c # v5.0.2
- - name: Build Iceberg Open API project
- run: ./gradlew :iceberg-open-api:shadowJar
+ - name: Build REST server and cloud bundle JARs
+ run: |
+ ./gradlew \
+ :iceberg-open-api:installDist \
Review Comment:
The workflow invokes only `:iceberg-open-api:stageRESTFixtureRuntime`, not
every project’s `shadowJar`. Following the Kafka Connect distribution, the
fixture now copies the cloud bundles’ runtime dependency declarations as loose
SDK JARs without shipping the shaded cloud bundle JARs. This also fixes the
mixed-classpath Azure linkage error found during review. I checked the updated
task graph: the only required shadow task is bundled Guava. The OpenAPI shaded
runtime task is removed, and staging remains outside the normal OpenAPI build.
Backend dependency pruning remains a separate follow-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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
