Skip to content

Conversation

@NguyenHoangSon96
Copy link

@NguyenHoangSon96 NguyenHoangSon96 commented Dec 22, 2025

Proposed Changes

  • Our project works with a library called arrow-java, and this library calls OkHttpReadableBuffer.readBytes(ByteBuffer dest) but this method was not implmented for grpc-okhttp so the calls failed. So I implemented this method so our project can work with arrow-java.
  • I have a talk with the team why this method is not implemented Here

Changes

  • Implement OkHttpReadableBuffer.readBytes(ByteBuffer dest).
  • Make readToByteBufferShouldSucceed test runs for OkHttpReadableBuffer.

Additional Information

  • Codecov complains about missing a test for throw new RuntimeException(e), I'm not sure if this line need to be tested.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Dec 22, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: NguyenHoangSon96 / name: sonnh (dbb5403)

@NguyenHoangSon96 NguyenHoangSon96 changed the title [EMPTY] trigger CI feat: support readBytes(ByteBuffer dest) for OkHttpReadableBuffer Dec 22, 2025
@NguyenHoangSon96 NguyenHoangSon96 marked this pull request as ready for review December 22, 2025 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant