Title: GH-139: [Flight] Stop return null from MetadataAdapter.getAll(String) and getAllByte(String) by axreldable · Pull Request #1016 · apache/arrow-java · GitHub
Open Graph Title: GH-139: [Flight] Stop return null from MetadataAdapter.getAll(String) and getAllByte(String) by axreldable · Pull Request #1016 · apache/arrow-java
X Title: GH-139: [Flight] Stop return null from MetadataAdapter.getAll(String) and getAllByte(String) by axreldable · Pull Request #1016 · apache/arrow-java
Description: What's Changed CallHeaders has 3 implementations: FlightCallHeaders ErrorFlightMetadata MetadataAdapter Before this change: MetadataAdapter could return null from getAll(String) and getAllByte(String) when there were no values for the key, because gRPC’s Metadata.getAll() returns null in that case. This was undocumented and forced callers to null-check. After this change: All 3 implementations consistently return an empty iterable (never null) when the key is absent or has no values. The contract is documented on the interface and covered by tests for each implementation. This contains breaking changes. MetadataAdapter.getAll(String) and getAllByte(String) return empty iterator instead of null. Closes #139.
Open Graph Description: What's Changed CallHeaders has 3 implementations: FlightCallHeaders ErrorFlightMetadata MetadataAdapter Before this change: MetadataAdapter could return null from getAll(String) and getAllByt...
X Description: What's Changed CallHeaders has 3 implementations: FlightCallHeaders ErrorFlightMetadata MetadataAdapter Before this change: MetadataAdapter could return null from getAll(String) and getAl...
Opengraph URL: https://github.com/apache/arrow-java/pull/1016
X: @github
Domain: patch-diff.githubusercontent.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:395d2891-2618-fc27-3425-64208e4f9f01 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 8A6A:281D19:302AD2:3E8137:69915E50 |
| html-safe-nonce | fa53cdfaae13f27faefd9089e95e05ce01e83a518916a3ef7ded5cd4803a7a53 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4QTZBOjI4MUQxOTozMDJBRDI6M0U4MTM3OjY5OTE1RTUwIiwidmlzaXRvcl9pZCI6Ijc3OTY0ODc5NzA1MDUxODY4OTYiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | bf8db500bb12bc75575637e35b47a8b953b2019f2c87ca818b42475fae73b412 |
| hovercard-subject-tag | pull_request:3264266290 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,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/apache/arrow-java/pull/1016/files |
| twitter:image | https://avatars.githubusercontent.com/u/20573448?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/20573448?s=400&v=4 |
| og:image:alt | What's Changed CallHeaders has 3 implementations: FlightCallHeaders ErrorFlightMetadata MetadataAdapter Before this change: MetadataAdapter could return null from getAll(String) and getAllByt... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 42c603b9d642c4a9065a51770f75e5e27132fef0e858607f5c9cb7e422831a7b |
| turbo-cache-control | no-preview |
| diff-view | unified |
| 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 full-width |
| disable-turbo | true |
| 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