fallintoplace opened a new pull request, #1141:
URL: https://github.com/apache/arrow-go/pull/1141

   ### Rationale for this change
   
   `recordEncoder.encode` ignored errors returned by body compression, so a 
failed compressor could still result in record metadata being built and 
returned as a successful encode.
   
   ### What changes are included in this PR?
   
   Return compression errors from `recordEncoder.encode`, release the temporary 
compressed buffer on failure, and add a regression test with a failing 
compressor.
   
   ### Are these changes tested?
   
   - `go test ./arrow/ipc`
   
   ### Are there any user-facing changes?
   
   IPC record encoding now reports compressor failures to the caller.


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