Skip to content

HDDS-15287. Remove unused deferred/async RPC#10280

Merged
adoroszlai merged 2 commits into
apache:masterfrom
adoroszlai:HDDS-15287
May 19, 2026
Merged

HDDS-15287. Remove unused deferred/async RPC#10280
adoroszlai merged 2 commits into
apache:masterfrom
adoroszlai:HDDS-15287

Conversation

@adoroszlai
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Remove unused code for deferred response and async mode Hadoop RPC.

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

How was this patch tested?

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

@adoroszlai adoroszlai self-assigned this May 15, 2026
@adoroszlai adoroszlai added the code-cleanup Changes that aim to make code better, without changing functionality. label May 15, 2026
@adoroszlai adoroszlai requested review from ivandika3 and szetszwo May 17, 2026 05:35
@adoroszlai
Copy link
Copy Markdown
Contributor Author

Before:
1-before

After:
2-after

Copy link
Copy Markdown

@yandrey321 yandrey321 left a comment

Choose a reason for hiding this comment

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

lgtm

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.

LGTM +1.

Copy link
Copy Markdown
Contributor

@szetszwo szetszwo left a comment

Choose a reason for hiding this comment

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

+1 the change looks good.

@adoroszlai adoroszlai merged commit 05e45f3 into apache:master May 19, 2026
47 of 48 checks passed
@adoroszlai
Copy link
Copy Markdown
Contributor Author

Thanks @ivandika3, @szetszwo, @yandrey321 for the review.

@adoroszlai adoroszlai deleted the HDDS-15287 branch May 19, 2026 06:15
adoroszlai added a commit that referenced this pull request May 19, 2026
@adoroszlai
Copy link
Copy Markdown
Contributor Author

Sorry, I had to revert this due to compile error on master (conflict with 526eec0). I will open new PR with these changes + small additional commit, but have to wait for #10285 to be merged, since that will also conflict with this.

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants