90degs2infty commented on issue #16805: URL: https://github.com/apache/datafusion/issues/16805#issuecomment-3150877557
> What do you think? I guess if `object_store` is treated as part of DataFusion's public API and this is a deliberate/concious decision, then exposing its types to the user is perfectly fine. As you can probably tell from the issue itself, I'm rather new to DataFusion and not super familiar with DataFusion's public API surface in general. Having just read on the design of error types, I might have been a little biased when opening the issue - sorry about that. > 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: This would also help users in places where DataFusion expects things from the `object_store` crate as input (e.g. [`SessionContext::register_object_store`](https://docs.rs/datafusion/latest/datafusion/execution/context/struct.SessionContext.html#method.register_object_store)). -- 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
