alamb commented on PR #6635: URL: https://github.com/apache/arrow-rs/pull/6635#issuecomment-2444122630
> This is relatively harmless, but does strike me as a bit odd, one wouldn't normally derive Clone for Error types. There are various times I think the lack of Clone causes annoyances downstream for us: * (e.g. getting a `Box<dyn Error>` that we are downcasting to a concrete object store error to fulfill some other contract where we would be quite happy to pay the copy cost) * Trying to save the error for results analysis I realize the lack of Clone is not arrow-rs specific (it is Rust), but it strikes me as overly pedantic sometimes. Here is a related discusion: https://github.com/rust-lang/rust/issues/24135 -- 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]
