From e0cf2bd07a7ef7613e267effb9ab102110a6fc3f Mon Sep 17 00:00:00 2001 From: nlou9 Date: Wed, 29 Apr 2026 15:00:30 -0700 Subject: [PATCH 1/3] feat: import confluent-common-bom for dependency version management Replace individual dependencyManagement entries with a single confluent-common-bom import for third-party dependencies. This delegates version management to the BOM while keeping all properties for downstream repo backward compatibility. Deps removed (now managed by BOM): avro, classgraph, commons-io, commons-lang3, commons-beanutils, commons-codec, commons-compress, commons-validator, grpc-bom, okio-jvm, protobuf-java, snakeyaml, jetty-bom, snappy-java, jose4j, guava, httpclient5, slf4j-api, jaxb-api, spotbugs-annotations, bcpkix-jdk18on, bcprov-jdk18on, bc-fips, bctls-fips, bcpkix-fips, bcutil-fips, gson, netty-bom, slf4j-reload4j, logback-core, reload4j, logredactor, junit, easymock, powermock-*. Deps kept (not in BOM): jackson-bom, aws-java-sdk-bom, aws-sdk-v2, azure-*, scala-*, kafka-*, confluent internal modules, junit-bom, mockito-bom, mockito-all, hamcrest-all, log4j-slf4j-impl (runtime scope). Co-Authored-By: Claude Opus 4.6 --- pom.xml | 262 +++++--------------------------------------------------- 1 file changed, 21 insertions(+), 241 deletions(-) diff --git a/pom.xml b/pom.xml index 40615a0069..8c80018f19 100644 --- a/pom.xml +++ b/pom.xml @@ -232,117 +232,24 @@ - + - org.apache.avro - avro - ${avro.version} - - - - io.github.classgraph - classgraph - ${classgraph.version} - - - - commons-io - commons-io - ${commons-io.version} - - - - org.apache.commons - commons-lang3 - ${commons-lang3.version} - - - - commons-beanutils - commons-beanutils - ${commons-beanutils.version} - - - commons-codec - commons-codec - ${commons-codec.version} - - - org.apache.commons - commons-compress - ${commons-compress.version} - - - - commons-validator - commons-validator - ${commons-validator.version} - - - - io.grpc - grpc-bom - 1.75.0 - pom - import - - - - com.squareup.okio - okio-jvm - ${okio.version} - - - - com.google.protobuf - protobuf-java - ${protobuf.version} - - - - org.yaml - snakeyaml - ${snakeyaml.version} - - - - org.eclipse.jetty - jetty-bom - ${jetty.version} + io.confluent + confluent-common-bom + 0.0.1-SNAPSHOT pom import - - - org.xerial.snappy - snappy-java - ${snappy.version} - - - - org.bitbucket.b_c - jose4j - ${jose4j.version} - - + + - com.google.guava - guava - ${guava.version} + com.fasterxml.jackson + jackson-bom + ${jackson.version} + import + pom - + com.amazonaws aws-java-sdk-bom @@ -357,6 +264,7 @@ pom import + com.azure azure-identity @@ -372,73 +280,16 @@ azure-security-keyvault-keys ${azure-security-keyvault-keys.version} - - - org.apache.httpcomponents.client5 - httpclient5 - ${httpclient5.version} - - - org.slf4j - slf4j-api - ${slf4j.version} - - - javax.xml.bind - jaxb-api - ${jaxb.version} - - - com.github.spotbugs - spotbugs-annotations - ${spotbugs.version} - - - org.bouncycastle - bcpkix-jdk18on - ${bouncycastle.jdk18.version} - - - org.bouncycastle - bcprov-jdk18on - ${bouncycastle.jdk18.version} - - - org.bouncycastle - bc-fips - ${bouncycastle.fips.version} - - - org.bouncycastle - bctls-fips - ${bouncycastle.tls-fips.version} - - - org.bouncycastle - bcpkix-fips - ${bouncycastle.bcpkix-fips.version} - - - org.bouncycastle - bcutil-fips - ${bouncycastle.bcutil-fips.version} - - - com.google.code.gson - gson - ${gson.version} - - + - com.fasterxml.jackson - jackson-bom - ${jackson.version} - import - pom + org.apache.logging.log4j + log4j-slf4j-impl + runtime + ${log4j2.version} - + + org.scala-lang scala-library @@ -497,42 +348,7 @@ connect-file ${kafka.version} - - org.slf4j - slf4j-reload4j - ${slf4j-reload4j.version} - - - - ch.qos.logback - logback-core - ${logback-core.version} - - - org.apache.logging.log4j - log4j-slf4j-impl - runtime - ${log4j2.version} - - - ch.qos.reload4j - reload4j - ${reload4j.version} - - - io.confluent - logredactor - ${logredactor.version} - - - io.netty - netty-bom - ${netty.version} - import - pom - + io.confluent @@ -551,12 +367,6 @@ - - junit - junit - ${junit.version} - test - org.junit junit-bom @@ -564,36 +374,6 @@ pom import - - org.easymock - easymock - ${easymock.version} - test - - - org.powermock - powermock-module-junit4 - ${powermock.version} - test - - - org.powermock - powermock-core - ${powermock.version} - test - - - org.powermock - powermock-api-easymock - ${powermock.version} - test - - - org.powermock - powermock-api-mockito2 - ${powermock.version} - test - org.mockito mockito-bom From fb24a18fde94063a999d4e8fc456b41584a89b2e Mon Sep 17 00:00:00 2001 From: nlou9 Date: Wed, 29 Apr 2026 15:31:24 -0700 Subject: [PATCH 2/3] fix: add confluent-common-bom snapshot repository Add CodeArtifact repository entry so Maven can resolve confluent-common-bom:0.0.1-SNAPSHOT during CI builds. Co-Authored-By: Claude Opus 4.6 --- pom.xml | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/pom.xml b/pom.xml index 8c80018f19..ca78cb104d 100644 --- a/pom.xml +++ b/pom.xml @@ -221,6 +221,13 @@ confluent https://packages.confluent.io/maven/ + + confluent-common-bom-snapshots + https://confluent-519856050701.d.codeartifact.us-west-2.amazonaws.com/maven/maven-snapshots/ + + true + + From eda1693c874016447ff7e636f897ae8afcb21060 Mon Sep 17 00:00:00 2001 From: nlou9 Date: Wed, 29 Apr 2026 15:43:02 -0700 Subject: [PATCH 3/3] revert: remove internal CodeArtifact repo entry from public pom.xml The BOM resolution needs to be handled via CI Maven settings, not in the public pom.xml. Co-Authored-By: Claude Opus 4.6 --- pom.xml | 7 ------- 1 file changed, 7 deletions(-) diff --git a/pom.xml b/pom.xml index ca78cb104d..8c80018f19 100644 --- a/pom.xml +++ b/pom.xml @@ -221,13 +221,6 @@ confluent https://packages.confluent.io/maven/ - - confluent-common-bom-snapshots - https://confluent-519856050701.d.codeartifact.us-west-2.amazonaws.com/maven/maven-snapshots/ - - true - -