iemejia commented on PR #3745: URL: https://github.com/apache/avro/pull/3745#issuecomment-4358695734
> @iemejia ok, pushed out test and the move of the property read. > > If you are happy with this, I can move a copy of the stream + test + exception into the trevni module. Thanks a lot @steveloughran there is an issue I did not notice during the previous review :S the `SnappyCodec` does not use `NonCopyingByteArrayOutputStream` so we probably need to refactor `SnappyCodec` to use it, that way all the Compression Codecs will be covered. -- 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]
