alamb commented on issue #16805:
URL: https://github.com/apache/datafusion/issues/16805#issuecomment-3141407372

   > Thanks for raising this. DataFusionError::ObjectStore does indeed wrap 
object_store::Error directly, and the crate exposes several types such as 
ObjectStoreUrl. That means the object_store crate is effectively part of 
DataFusion’s public API
   
   
   Yes I agree with this
   
   Something we could do to make it easier on downstream crates would be to 
publically re-export the whole object store crate like we do for arrow: 
   
   
https://github.com/apache/datafusion/blob/2a90ff606d697ca77c67da2ce72e40f2e67119e3/datafusion/common/src/lib.rs#L63
   
   Something like
   
   ```rust
   pub use object_store
   ```
   
   That way users could just import anything from object_store like `use 
datafusion::object_store::ObjectStoreError`


-- 
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-unsubscr...@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to