You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/05/03 20:34:04 UTC

[GitHub] [arrow-datafusion] jdye64 opened a new issue, #2436: `ContextProvider`: custom message for error propagation

jdye64 opened a new issue, #2436:
URL: https://github.com/apache/arrow-datafusion/issues/2436

   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   The current implementation of `ContextProvider::get_table_provider()` return an `Option`. Normally this is fine, but with custom `ContextProvider's` a user would prefer to define their own messages that are encapsulated in a `DataFusionError`.
   
   **Describe the solution you'd like**
   Alter `ContextProvider::get_table_provider()` to return a `Result<Arc<dyn table::TableProvider>, DataFusionError>` instead of an `Option<Arc<dyn table::TableProvider>>` and any necessary changes around that.
   
   **Describe alternatives you've considered**
   None
   
   **Additional context**
   None
   


-- 
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: github-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow-datafusion] andygrove closed issue #2436: `ContextProvider`: custom message for error propagation

Posted by GitBox <gi...@apache.org>.
andygrove closed issue #2436: `ContextProvider`: custom message for error propagation
URL: https://github.com/apache/arrow-datafusion/issues/2436


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org