alamb commented on issue #22863: URL: https://github.com/apache/datafusion/issues/22863#issuecomment-4671706429
> ### Is your feature request related to a problem or challenge? > The current categories under DataFusionError is not Mutually Exclusive, Collectively Exhaustive (MECE), making it hard to use consistently. > > For example, if the user is developing a custom Object Store, should they return DataFusionError::ObjectStore, DataFusionError::IoError, or DataFusionError::External? I would expect `DataFusionError::ObjectStore` > Similarly, for an error happened during the plan stage of a custom TableProvider, should that be DataFusionError::Plan, or DataFusionError::External? I would suggest `DataFusionError::Plan` -- and use the `source` field when needed 🤔 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
