incubator-uniffle icon indicating copy to clipboard operation
incubator-uniffle copied to clipboard

[Improvement] Remove the ShuffleBlockInfo used in the communication between client and server via handleSendShuffleDataRequest.

Open lwllvyb opened this issue 1 year ago • 0 comments

Code of Conduct

Search before asking

  • [X] I have searched in the issues and found no similar issues.

What would you like to be improved?

  1. Remove the ShuffleBlockInfo used in the communication between client and server via handleSendShuffleDataRequest.
  2. The code byteBuf.writeInt(shuffleBlockInfo.getLength()); in encodeShuffleBlockInfo is redundant, because the block data length has been encoded in shuffleBlockInfo.copyDataTo(byteBuf)

Related PR: #2181

How should we improve?

No response

Are you willing to submit PR?

  • [X] Yes I am willing to submit a PR!

lwllvyb avatar Oct 18 '24 09:10 lwllvyb