Arawoof06 commented on PR #1318: URL: https://github.com/apache/arrow-java/pull/1318#issuecomment-6005846836
Pushed fae1400 addressing the review. The `available()` bound is now limited to `KnownLength` streams and everything else is read incrementally with the buffer allocated after the bytes arrive, `frame()` releases `appMetadata`/`body` on failure and on a repeated field, the length read and check are in one helper, and the messages are split and no longer go through `String.format`. The tests are parameterized over gRPC's `BufferInputStream` and a `GZIPInputStream` and cover the negative, oversized, leak and repeated-field cases on both. I also updated the description to match the new shape. -- 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]
