This is an automated email from the ASF dual-hosted git repository.
lhotari pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git
The following commit(s) were added to refs/heads/master by this push:
new c9b893f58f Remove commons-configuration2 and commons-beanutils from
top level dependencies (#4648)
c9b893f58f is described below
commit c9b893f58fe200776f237110514785848a4cda80
Author: Lari Hotari <lhot...@users.noreply.github.com>
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 cd50d540f0..cc54811599 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 c05953ddd7..64f952e6f3 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 2faeda6037..5b65c7a092 100644
--- a/pom.xml
+++ b/pom.xml
@@ -128,6 +128,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.5</bouncycastle.version>
<curator.version>5.7.1</curator.version>
@@ -298,6 +299,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>
@@ -871,15 +877,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 3cd7712de3..0fe37cd32e 100644
--- a/stats/bookkeeper-stats-api/pom.xml
+++ b/stats/bookkeeper-stats-api/pom.xml
@@ -58,5 +58,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 11d3c8668b..fd995e8b4d 100644
--- a/tests/shaded/distributedlog-core-shaded-test/pom.xml
+++ b/tests/shaded/distributedlog-core-shaded-test/pom.xml
@@ -79,6 +79,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>
