AngersZhuuuu commented on PR #3171:
URL: https://github.com/apache/celeborn/pull/3171#issuecomment-2760959852
> @AngersZhuuuu Maybe we can just modify
org/apache/celeborn/client/ShuffleClientImpl:PushMergedDataWrappedCallback
>
> ```
> @Override
> public void onSuccess(ByteBuffer response) {
> byte reason = response.get();
> ......
> ```
>
> to
>
> ```
> @Override
> public void onSuccess(ByteBuffer response) {
> byte reason = StatusCode.SUCCESS.getValue();
> if (response.remaining() > 0) {
> reason = response.get();
> }
> ......
> ```
Since new client version not support older version worker, after discuss, we
decided to remove this code. cc @RexXiong
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
