HDDS-15205. Cut WritableRPCEngine#10213
Conversation
ivandika3
left a comment
There was a problem hiding this comment.
LGTM +1. Same as HADOOP-19764 but without additional comments.
|
Thanks @ivandika3 for the review. I have noticed a follow-up PR (apache/hadoop#8470), included in another commit here. |
|
@adoroszlai The implementation looks good and there is already a +1 so I think it's ok to include in this patch. I'm still +1. |
|
@adoroszlai I think we should include the test In the future Ozone might decide to change the Hadoop RPC logic to suite Ozone's need so we need test to catch regressions. |
Unit tests for Hadoop RPC are not included in Ozone, because RPC was copied from Hadoop 3.2, the last version without We can attempt to copy and migrate tests in a follow-up. |
|
@adoroszlai Got it, thanks for the info. |
What changes were proposed in this pull request?
Port from Hadoop: HADOOP-19864. Cut WritableRPCEngine.
Ozone uses
ProtobufRpcEngine, never usedWritableRPCEngine.https://issues.apache.org/jira/browse/HDDS-15205
How was this patch tested?
https://github.com/adoroszlai/ozone/actions/runs/25539302248