This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch branch-4.17 in repository https://gitbox.apache.org/repos/asf/bookkeeper.git
commit da18f928bdadf5859b912353e8f94668e07f8ca0 Author: Lari Hotari <[email protected]> AuthorDate: Tue Aug 12 11:38:43 2025 +0300 Remove commons-configuration2 and commons-beanutils from top level dependencies (#4648) * Remove commons-configuration2 and commons-beanutils from dependencies of all modules * Add commons-logging to dependency management --- bookkeeper-common/pom.xml | 9 +++++++++ bookkeeper-http/http-server/pom.xml | 9 +++++++++ pom.xml | 15 ++++++--------- stats/bookkeeper-stats-api/pom.xml | 9 +++++++++ tests/shaded/distributedlog-core-shaded-test/pom.xml | 9 +++++++++ 5 files changed, 42 insertions(+), 9 deletions(-) diff --git a/bookkeeper-common/pom.xml b/bookkeeper-common/pom.xml index dc29a74845..c5690b2532 100644 --- a/bookkeeper-common/pom.xml +++ b/bookkeeper-common/pom.xml @@ -35,6 +35,15 @@ <artifactId>cpu-affinity</artifactId> <version>${project.parent.version}</version> </dependency> + <dependency> + <groupId>org.apache.commons</groupId> + <artifactId>commons-configuration2</artifactId> + </dependency> + <!-- commons-configuration2 has an optional dependency on commons-beanutils --> + <dependency> + <groupId>commons-beanutils</groupId> + <artifactId>commons-beanutils</artifactId> + </dependency> <dependency> <groupId>com.google.guava</groupId> <artifactId>guava</artifactId> diff --git a/bookkeeper-http/http-server/pom.xml b/bookkeeper-http/http-server/pom.xml index 206f31cad2..f91184b354 100644 --- a/bookkeeper-http/http-server/pom.xml +++ b/bookkeeper-http/http-server/pom.xml @@ -28,6 +28,15 @@ <name>Apache BookKeeper :: Http :: Http Server</name> <url>http://maven.apache.org</url> <dependencies> + <dependency> + <groupId>org.apache.commons</groupId> + <artifactId>commons-configuration2</artifactId> + </dependency> + <!-- commons-configuration2 has an optional dependency on commons-beanutils --> + <dependency> + <groupId>commons-beanutils</groupId> + <artifactId>commons-beanutils</artifactId> + </dependency> <dependency> <groupId>com.fasterxml.jackson.core</groupId> <artifactId>jackson-core</artifactId> diff --git a/pom.xml b/pom.xml index 11e24aaf30..bedd4a3c25 100644 --- a/pom.xml +++ b/pom.xml @@ -126,6 +126,7 @@ <commons-compress.version>1.27.1</commons-compress.version> <commons-lang.version>2.6</commons-lang.version> <commons-lang3.version>3.17.0</commons-lang3.version> + <commons-logging.version>1.3.5</commons-logging.version> <commons-io.version>2.19.0</commons-io.version> <bouncycastle.version>1.0.2.4</bouncycastle.version> <curator.version>5.7.1</curator.version> @@ -292,6 +293,11 @@ <artifactId>commons-lang</artifactId> <version>${commons-lang.version}</version> </dependency> + <dependency> + <groupId>commons-logging</groupId> + <artifactId>commons-logging</artifactId> + <version>${commons-logging.version}</version> + </dependency> <dependency> <groupId>com.google.guava</groupId> <artifactId>guava</artifactId> @@ -865,15 +871,6 @@ <groupId>org.slf4j</groupId> <artifactId>slf4j-api</artifactId> </dependency> - <dependency> - <groupId>org.apache.commons</groupId> - <artifactId>commons-configuration2</artifactId> - </dependency> - <!-- commons-configuration2 has an optional dependency on commons-beanutils --> - <dependency> - <groupId>commons-beanutils</groupId> - <artifactId>commons-beanutils</artifactId> - </dependency> <!-- test dependencies --> <dependency> <groupId>junit</groupId> diff --git a/stats/bookkeeper-stats-api/pom.xml b/stats/bookkeeper-stats-api/pom.xml index 17e2c51b57..e5f35818f0 100644 --- a/stats/bookkeeper-stats-api/pom.xml +++ b/stats/bookkeeper-stats-api/pom.xml @@ -59,5 +59,14 @@ </plugins> </build> <dependencies> + <dependency> + <groupId>org.apache.commons</groupId> + <artifactId>commons-configuration2</artifactId> + </dependency> + <!-- commons-configuration2 has an optional dependency on commons-beanutils --> + <dependency> + <groupId>commons-beanutils</groupId> + <artifactId>commons-beanutils</artifactId> + </dependency> </dependencies> </project> diff --git a/tests/shaded/distributedlog-core-shaded-test/pom.xml b/tests/shaded/distributedlog-core-shaded-test/pom.xml index 6410639a5e..93cb501d82 100644 --- a/tests/shaded/distributedlog-core-shaded-test/pom.xml +++ b/tests/shaded/distributedlog-core-shaded-test/pom.xml @@ -75,6 +75,15 @@ <groupId>io.netty</groupId> <artifactId>netty-buffer</artifactId> </dependency> + <dependency> + <groupId>org.apache.commons</groupId> + <artifactId>commons-configuration2</artifactId> + </dependency> + <!-- commons-configuration2 has an optional dependency on commons-beanutils --> + <dependency> + <groupId>commons-beanutils</groupId> + <artifactId>commons-beanutils</artifactId> + </dependency> </dependencies> <build> <plugins>
