Skip to content

Build, Kafka-Connect: Disable publishing for empty grouping project#15496

Merged
bryanck merged 3 commits intoapache:mainfrom
dramaticlly:kafka-connect-duplicate-publish
Mar 3, 2026
Merged

Build, Kafka-Connect: Disable publishing for empty grouping project#15496
bryanck merged 3 commits intoapache:mainfrom
dramaticlly:kafka-connect-duplicate-publish

Conversation

@dramaticlly
Copy link
Contributor

@dramaticlly dramaticlly commented Mar 3, 2026

Running into publish issue with org.apache.iceberg:iceberg-kafka-connect as I realized there's name collision in top level and nested project. Disabled top level one here to allow a happy publish

CC @bryanck @amogh-jahagirdar @huaxingao

Local test to verify the new behavior

$ ./gradlew :iceberg-kafka-connect:publishToMavenLocal :iceberg-kafka-connect:iceberg-kafka-connect:publishToMavenLocal 2>&1 | grep -E "(publishApache|publishTo|SKIPPED|disabled)" 

> Task :iceberg-kafka-connect:generatePomFileForApachePublication SKIPPED
> Task :iceberg-kafka-connect:generateMetadataFileForApachePublication SKIPPED
> Task :iceberg-kafka-connect:publishApachePublicationToMavenLocal SKIPPED
> Task :iceberg-kafka-connect:publishToMavenLocal SKIPPED
> Task :iceberg-kafka-connect:iceberg-kafka-connect:publishApachePublicationToMavenLocal
> Task :iceberg-kafka-connect:iceberg-kafka-connect:publishToMavenLocal

@dramaticlly
Copy link
Contributor Author

also I think it will close #14549

@dramaticlly dramaticlly changed the title Build: Disable publishing for nested iceberg-kafka-connect subproject Build: Disable publishing for empty iceberg-kafka-connect grouping project Mar 3, 2026
@dramaticlly dramaticlly changed the title Build: Disable publishing for empty iceberg-kafka-connect grouping project Build, Kafka-Connect: Disable publishing for empty grouping project Mar 3, 2026
Copy link
Contributor

@huaxingao huaxingao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bryanck
Copy link
Contributor

bryanck commented Mar 3, 2026

LGTM, thanks for catching this @dramaticlly . Also, thanks for the review @huaxingao !

@bryanck bryanck merged commit 88d4604 into apache:main Mar 3, 2026
12 checks passed
@bryanck bryanck self-requested a review March 3, 2026 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants