Title: [FlightRPC] Remove sketchy gRPC reflection code · Issue #939 · apache/arrow-java · GitHub
Open Graph Title: [FlightRPC] Remove sketchy gRPC reflection code · Issue #939 · apache/arrow-java
X Title: [FlightRPC] Remove sketchy gRPC reflection code · Issue #939 · apache/arrow-java
Description: Describe the bug, including details regarding any error messages, version, and platform. grpc/grpc-java#12579 There are public methods to get ownership of the internal ByteBuffers, instead of having to use reflection (which, did we actua...
Open Graph Description: Describe the bug, including details regarding any error messages, version, and platform. grpc/grpc-java#12579 There are public methods to get ownership of the internal ByteBuffers, instead of havin...
X Description: Describe the bug, including details regarding any error messages, version, and platform. grpc/grpc-java#12579 There are public methods to get ownership of the internal ByteBuffers, instead of havin...
Opengraph URL: https://github.com/apache/arrow-java/issues/939
X: @github
Domain: patch-diff.githubusercontent.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"[FlightRPC] Remove sketchy gRPC reflection code","articleBody":"### Describe the bug, including details regarding any error messages, version, and platform.\n\nhttps://github.com/grpc/grpc-java/pull/12579\n\nThere are public methods to get ownership of the internal ByteBuffers, instead of having to use reflection (which, did we actually ever need it in the first place?)\n\n\u003e There's no need to dig into our internals for accessing the ByteBuffer (which is what GetReadableBuffer reports to do). We have zero-copy APIs. They can use [HasByteBuffer.getByteBuffer()](https://grpc.github.io/grpc-java/javadoc/io/grpc/HasByteBuffer.html)+InputStream.skip() to loop through the ByteBuffers. If they want to access all the byte buffers simultaneously (as skip() will deallocate the last byte buffer, just like read() does), then they can use InputStream.mark(). If they want to take over ownership of the ByteBuffers, then they can use [Detachable.detach()](https://grpc.github.io/grpc-java/javadoc/io/grpc/Detachable.html).","author":{"url":"https://github.com/lidavidm","@type":"Person","name":"lidavidm"},"datePublished":"2025-12-29T04:31:32.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/939/arrow-java/issues/939"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:df670b3d-91ac-a0df-dd76-ba1fb8dc4ddc |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | AB36:3F004:2A6F945:36FCBBE:6990CFB6 |
| html-safe-nonce | 64123a737307fbcb4d4e071d58080a1fca836f6c6718418733396ec7cb5e357d |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBQjM2OjNGMDA0OjJBNkY5NDU6MzZGQ0JCRTo2OTkwQ0ZCNiIsInZpc2l0b3JfaWQiOiI1ODY3OTI3NDk4NTk2MDc3NDk0IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | d70403151d203d79e18446589d839e4bd9f6b1bdbcf6248eea341efb43d9a68e |
| hovercard-subject-tag | issue:3766384236 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/apache/arrow-java/939/issue_layout |
| twitter:image | https://opengraph.githubassets.com/336b02e2efd08b04e8d075995f633cb1557e3bb0e9289544ec37a103aeaf5f5d/apache/arrow-java/issues/939 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/336b02e2efd08b04e8d075995f633cb1557e3bb0e9289544ec37a103aeaf5f5d/apache/arrow-java/issues/939 |
| og:image:alt | Describe the bug, including details regarding any error messages, version, and platform. grpc/grpc-java#12579 There are public methods to get ownership of the internal ByteBuffers, instead of havin... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | lidavidm |
| hostname | github.com |
| expected-hostname | github.com |
| None | 42c603b9d642c4a9065a51770f75e5e27132fef0e858607f5c9cb7e422831a7b |
| turbo-cache-control | no-preview |
| go-import | github.com/apache/arrow-java git https://github.com/apache/arrow-java.git |
| octolytics-dimension-user_id | 47359 |
| octolytics-dimension-user_login | apache |
| octolytics-dimension-repository_id | 893682219 |
| octolytics-dimension-repository_nwo | apache/arrow-java |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 893682219 |
| octolytics-dimension-repository_network_root_nwo | apache/arrow-java |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 848bc6032dcc93a9a7301dcc3f379a72ba13b96e |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width