Skip to content

HDDS-11717. Upgrade Spring Framework to 6.2.19, jOOQ to 3.19.37 - #11172

Open
adoroszlai wants to merge 2 commits into
apache:masterfrom
adoroszlai:HDDS-11717
Open

HDDS-11717. Upgrade Spring Framework to 6.2.19, jOOQ to 3.19.37#11172
adoroszlai wants to merge 2 commits into
apache:masterfrom
adoroszlai:HDDS-11717

Conversation

@adoroszlai

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Spring 5 is EOL, upgrade to current GA version 6.2.19. Also bump jOOQ to 3.19.37 (the last version with Java 17 support).

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

How was this patch tested?

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

@adoroszlai
adoroszlai requested a review from devmadhuu August 31, 2026 08:38
@adoroszlai adoroszlai self-assigned this Aug 31, 2026
@adoroszlai adoroszlai added the dependencies Pull requests that update a dependency file label Aug 31, 2026
private void insertGlobalStatsRecords(ReconGlobalStatsManager statsManager,
Timestamp timestamp, String key,
long value) throws IOException {
String key, long value) throws IOException {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: TestOmDBInsightEndPoint doesn't look like it needs changes for the jOOQ bump. The Timestamp param of insertGlobalStatsRecords was already unused (the body is just new GlobalStatsValue(value)), so dropping it is an independent cleanup rather than part of the version bump. No objection either way. I can open a small follow-up issue for it if you'd rather keep this PR to the upgrade.

@rich7420 rich7420 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

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

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants