Skip to content

HDDS-14023. Extract utility for resizing thread pool#9387

Merged
adoroszlai merged 2 commits intoapache:masterfrom
adoroszlai:HDDS-14023
Nov 27, 2025
Merged

HDDS-14023. Extract utility for resizing thread pool#9387
adoroszlai merged 2 commits intoapache:masterfrom
adoroszlai:HDDS-14023

Conversation

@adoroszlai
Copy link
Copy Markdown
Contributor

@adoroszlai adoroszlai commented Nov 27, 2025

What changes were proposed in this pull request?

Extract existing logic for changing thread pool size.

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

How was this patch tested?

$ mvn -am -pl :ozone-integration-test -Dtest='TestOzoneIntegrationNonHA' clean verify
...

$ grep HddsServerUtil 'hadoop-ozone/integration-test/target/surefire-reports/org.apache.ozone.test.NonHATests$DatanodeReconfiguration-output.txt' | cut -c51-
INFO  utils.HddsServerUtil (HddsServerUtil.java:setPoolSize(754)) - org.apache.hadoop.ozone.container.replication.ReplicationServer pool size decreased from 10 to 9
INFO  utils.HddsServerUtil (HddsServerUtil.java:setPoolSize(754)) - org.apache.hadoop.ozone.container.replication.ReplicationServer pool size increased from 9 to 10
INFO  utils.HddsServerUtil (HddsServerUtil.java:setPoolSize(754)) - org.apache.hadoop.ozone.container.common.statemachine.commandhandler.DeleteBlocksCommandHandler pool size decreased from 5 to 4
INFO  utils.HddsServerUtil (HddsServerUtil.java:setPoolSize(754)) - org.apache.hadoop.ozone.container.common.statemachine.commandhandler.DeleteBlocksCommandHandler pool size increased from 4 to 5

@adoroszlai adoroszlai requested a review from ivandika3 November 27, 2025 11:10
@adoroszlai adoroszlai self-assigned this Nov 27, 2025
@adoroszlai adoroszlai added the code-cleanup Changes that aim to make code better, without changing functionality. label Nov 27, 2025
@ivandika3 ivandika3 marked this pull request as ready for review November 27, 2025 15:09
ivandika3
ivandika3 approved these changes Nov 27, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@ivandika3 ivandika3 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 patch. LGTM +1.

I'll update #9385 after this is merged.

adoroszlai
adoroszlai commented Nov 27, 2025
View reviewed changes
Comment thread hadoop-hdds/framework/src/main/java/org/apache/hadoop/hdds/utils/HddsServerUtil.java Outdated
@adoroszlai adoroszlai merged commit a29e2b1 into apache:master Nov 27, 2025
43 checks passed
@adoroszlai adoroszlai deleted the HDDS-14023 branch November 27, 2025 18:10
@adoroszlai
Copy link
Copy Markdown
Contributor Author

adoroszlai commented Nov 27, 2025

Thanks @ivandika3, @sodonnel for the review.

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

Reviewers

@ivandika3 ivandika3 ivandika3 approved these changes

@sodonnel sodonnel sodonnel approved these changes

Assignees

@adoroszlai adoroszlai

Labels

code-cleanup Changes that aim to make code better, without changing functionality.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@adoroszlai @sodonnel @ivandika3