steveloughran commented on PR #3745: URL: https://github.com/apache/avro/pull/3745#issuecomment-4360497864
its got some big assumptions about how data is collected and use of sync() and is more confident than I'd be, even though it's often my code underneath. See the changed design * restored DataWriter semantics * explicit static `restrictedCapacityOutputStream()` method for the capacity limiting; reviewed all codecs for use * Moved the limit checks into SystemLimitException * Snappy codec does a direct invocation of the SystemLimitException check before it allocs its buffer, so shares the same safety check. Test for that too. -- 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]
