Skip to content

HDDS-14917. Remove unused version definitions#9998

Merged
adoroszlai merged 4 commits intoapache:masterfrom
adoroszlai:HDDS-14917
Mar 31, 2026
Merged

HDDS-14917. Remove unused version definitions#9998
adoroszlai merged 4 commits intoapache:masterfrom
adoroszlai:HDDS-14917

Conversation

@adoroszlai
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Remove some unused version definitions:

  • Apache Commons components used only transitively, not directly by Ozone
  • com.codahale.metrics (predecessor of io.dropwizard.metrics)
  • declared.hadoop.version (leftover from Hadoop)
  • Hamcrest

https://issues.apache.org/jira/browse/HDDS-14917

How was this patch tested?

https://github.com/adoroszlai/ozone/actions/runs/23704952099

@adoroszlai adoroszlai self-assigned this Mar 29, 2026
@adoroszlai adoroszlai added dependencies Pull requests that update a dependency file code-cleanup Changes that aim to make code better, without changing functionality. labels Mar 29, 2026
Copy link
Copy Markdown
Contributor

@ptlrs ptlrs left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @adoroszlai. LGTM

Copy link
Copy Markdown
Contributor

@dombizita dombizita left a comment

Choose a reason for hiding this comment

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

Thanks @adoroszlai!

@adoroszlai adoroszlai merged commit 4a3c626 into apache:master Mar 31, 2026
86 of 87 checks passed
@adoroszlai adoroszlai deleted the HDDS-14917 branch March 31, 2026 14:47
@adoroszlai
Copy link
Copy Markdown
Contributor Author

Thanks @dombizita, @ptlrs for the review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-cleanup Changes that aim to make code better, without changing functionality. dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants