alexandreyc opened a new pull request, #4726:
URL: https://github.com/apache/arrow-rs/pull/4726
# Which issue does this PR close?
Does not close any open issue.
# Rationale for this change
I need to handle some kind of I/O errors (notably broken pipe errors)
specifically and the current `IoError` variant does not permit it because it
does not expose the underlying `std::io::Error`. Moreover, it looks to me that
`IoError` is a bit misused, mostly in the IPC format and Flight crates. I
introduced a new `IpcError` variant to have a better semantics of errors.
# What changes are included in this PR?
- Add `std::io::Error` field to `IoError` variant
- Add `IpcError` variant to `ArrowError`
- Update conversion traits
- Replace usage of `IoError` within the IPC / Flights crates
I'm not 100% sure about the new semantics of every errors so feel to correct
me.
# Are there any user-facing changes?
Yes. Callers that handle `IoError` separately will need to update their
patterns to include the new field and they will also need to handle the new
`IpcError` variant.
--
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]
