alamb commented on issue #11468: URL: https://github.com/apache/arrow-rs/issues/11468#issuecomment-6084860434
> Return ArrowError::NotYetImplemented consistently for unsupported type conversions, reserving CastError for failures when converting input values. I recommend this approach as it will be backwards compatible and we can release it with a minor version It will be somewhat misleading as there are some operations that will likely never be supported (e.g. cast from Struct to Decimal or something) as they don't make semantic sense, but I do think it is accurate that they are "NotYetSupported") Thank you for this nicely written issue > Add a structured kind to CastError, allowing callers to distinguish reasons such as UnsupportedConversion and InvalidValue while retaining the diagnostic message. This is probably cleaner and has more information, but if we did this it would have to wait for a major arrow release and would be much more disruptive to downstream crates / users I think -- 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]
