Skip to content

[8.4] Do not deserialize back BytesTransportRequest to clone a request in MockTransportService (#89926)#89936

Merged
fcofdez merged 1 commit intoelastic:8.4from
fcofdez:backport/8.4/pr-89926
Sep 8, 2022
Merged

[8.4] Do not deserialize back BytesTransportRequest to clone a request in MockTransportService (#89926)#89936
fcofdez merged 1 commit intoelastic:8.4from
fcofdez:backport/8.4/pr-89926

Conversation

@fcofdez
Copy link
Copy Markdown
Contributor

@fcofdez fcofdez commented Sep 8, 2022

Backports the following commits to 8.4:

…ockTransportService (elastic#89926)

Some handlers, such as JOIN_VALIDATE_ACTION_NAME, deserialize a
BytesTransportRequest into a different class that cannot be serialized.
This commit changes that behaviour and directly clones the BytesTransportRequest
instead of relying on the transport handler.

Closes elastic#88120
@fcofdez fcofdez added :Distributed/Network Http and internode communication implementations >test Issues or PRs that are addressing/adding tests backport Team:Distributed Meta label for distributed team. labels Sep 8, 2022
@fcofdez fcofdez merged commit ba317b4 into elastic:8.4 Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport :Distributed/Network Http and internode communication implementations Team:Distributed Meta label for distributed team. >test Issues or PRs that are addressing/adding tests v8.4.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants