Skip to content

Remove commons-configuration2 and commons-beanutils from top level dependencies#4648

Merged
lhotari merged 2 commits into
apache:masterfrom
lhotari:lh-remove-commons-configuration2-at-top-level
Aug 12, 2025
Merged

Remove commons-configuration2 and commons-beanutils from top level dependencies#4648
lhotari merged 2 commits into
apache:masterfrom
lhotari:lh-remove-commons-configuration2-at-top-level

Conversation

@lhotari

@lhotari lhotari commented Aug 7, 2025

Copy link
Copy Markdown
Member

Fix #4647

Motivation

See #4647

Changes

Move the commons-configuration2 and commons-beanutils from top level dependencies to the modules where they are required.

@lhotari lhotari added this to the 4.18.0 milestone Aug 7, 2025
@lhotari lhotari self-assigned this Aug 7, 2025
@StevenLuMT

Copy link
Copy Markdown
Member

rerun failure checks

@StevenLuMT StevenLuMT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good jobs

@lhotari
lhotari merged commit c9b893f into apache:master Aug 12, 2025
85 of 99 checks passed
merlimat pushed a commit that referenced this pull request Aug 21, 2025
…pendencies (#4648)

* Remove commons-configuration2 and commons-beanutils from dependencies of all modules

* Add commons-logging to dependency management
priyanshu-ctds pushed a commit to datastax/bookkeeper that referenced this pull request Aug 29, 2025
…pendencies (apache#4648)

* Remove commons-configuration2 and commons-beanutils from dependencies of all modules

* Add commons-logging to dependency management

(cherry picked from commit da18f92)
srinath-ctds pushed a commit to datastax/bookkeeper that referenced this pull request Aug 29, 2025
…pendencies (apache#4648)

* Remove commons-configuration2 and commons-beanutils from dependencies of all modules

* Add commons-logging to dependency management

(cherry picked from commit da18f92)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

3 participants