Skip to content

Conversation

@xborder
Copy link
Contributor

@xborder xborder commented Jan 11, 2026

What's Changed

  • The branch removes the previous reflection-based access to internal gRPC buffers and replaces it with supported gRPC interfaces (Detachable, HasByteBuffer) to safely transfer ownership of direct buffers to Arrow without copying.
  • gRPC direct ByteBuffer streams are detached and wrapped as ForeignAllocation for zero‑copy reads
  • Fall back to safe buffer copies when gRPC does not expose a direct ByteBuffer or data is fragmented
  • Added test coverage

Closes #939.

@github-actions

This comment has been minimized.

@xborder xborder changed the title GH-939: Remove relection GH-939: Remove reflection for gRPC buffers Jan 11, 2026
@xborder xborder marked this pull request as draft January 11, 2026 00:47
@lidavidm lidavidm added the enhancement PRs that add or improve features. label Jan 11, 2026
@xborder xborder marked this pull request as ready for review January 11, 2026 15:02
@github-actions github-actions bot added this to the 19.0.0 milestone Jan 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement PRs that add or improve features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FlightRPC] Remove sketchy gRPC reflection code

2 participants