Skip to content

[Bug] All modules have a direct dependency on commons-configuration2 and commons-beanutils which doesn't make sense #4647

Description

@lhotari

BUG REPORT

Describe the bug

All modules have a direct dependency on commons-configuration2 and commons-beanutils which doesn't make sense

bookkeeper/pom.xml

Lines 869 to 882 in 45f7a92

<!-- compilation dependencies (available at all classpaths) -->
<dependency>
<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>

Expected behavior

commons-configuration2 and commons-beanutils should be a dependency in the module that import the commons-configuration2 classes (commons-beanutils is an optional commons-configuration2 dependency):

The modules importing commons-configuration2 classes are:

bookkeeper-common
bookkeeper-http/http-server
bookkeeper-server
metadata-drivers/etcd
stats/bookkeeper-stats-api
stats/bookkeeper-stats-providers/codahale-metrics-provider
stats/bookkeeper-stats-providers/otel-metrics-provider
stats/bookkeeper-stats-providers/prometheus-metrics-provider
stream/distributedlog/common
stream/distributedlog/core
stream/distributedlog/io/dlfs
stream/server
stream/storage/api
stream/storage/impl
tools/framework
tools/ledger
tools/perf
tools/stream

Additional context

Related Pulsar issue is apache/pulsar#24611

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions