Skip to content

Commit 4c418e4

Browse files
Added uber minimum pom config in central publish (#1028)
Fix: Use minimal POM for Maven Central publishing Restores the original behavior where both main and thin JARs are published with the minimal POM (uber-minimal-pom.xml) instead of the full POM with all dependencies. Problem: Since v1.0.8, the main JAR was being published with the full POM (all dependencies) Only the thin JAR was using the minimal POM This caused dependency conflicts and larger POM files for users Solution: Add pomFile configuration to central-publishing-maven-plugin Ensures both main and thin JARs use the minimal POM for publishing ## Testing ## Additional Notes to the Reviewer --------- Signed-off-by: Nikhil Suri <nikhil.suri@databricks.com>
1 parent 5b9f54f commit 4c418e4

2 files changed

Lines changed: 3 additions & 0 deletions

File tree

NEXT_CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,5 +13,6 @@
1313
- Fixed state leaking issue in thrift client.
1414
- Fixed timestamp values returning only milliseconds instead of the full nanosecond precision.
1515
- Fixed Statement.getUpdateCount() for DML queries.
16+
- Fixed Maven Central publishing to use minimal POM for both main and thin JARs, restoring behavior from v1.0.7 and earlier
1617
---
1718
*Note: When making changes, please add your change under the appropriate section with a brief description.*

pom.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -698,6 +698,8 @@
698698
<publishingServerId>central</publishingServerId>
699699
<autoPublish>true</autoPublish>
700700
<waitUntil>published</waitUntil>
701+
<!-- Use minimal POM for fat jar publishing -->
702+
<pomFile>${project.basedir}/uber-minimal-pom.xml</pomFile>
701703
</configuration>
702704
</plugin>
703705
</plugins>

0 commit comments

Comments
 (0)