axreldable opened a new pull request, #1016:
URL: https://github.com/apache/arrow-java/pull/1016

   ## 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.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to