ion-elgreco commented on PR #348:
URL: 
https://github.com/apache/arrow-rs-object-store/pull/348#issuecomment-2876174998

   > I think I was misunderstanding how the client usage here might look, the 
registry contains _initialized_ and configured `ObjectStoreRef`s based on the 
provided `Url`. I was crossing the streams in my brain and thinking about the 
Url _scheme_ too much.
   > 
   > Upon further consideration this would be a bit more limiting than I 
initially considered. As such it would not replace the factories within 
delta-rs since there we are needing to register a 
factory/closure/whatever-you-wanna-call-it which will produce those 
`ObjectStoreRef` objects rather than caching initialized refs in some global 
state.
   > 
   > I'm pondering this a bit more but the visage of an API isn't coming to me 
yet 🤔
   
   We do have this actually: 
https://github.com/delta-io/delta-rs/blob/3d2bbef5cb7c5f6b7e5d1b778edbfd098e993bc3/crates/core/src/logstore/storage/mod.rs#L23
 (I grabbed that from datafusion codebase some time ago for the lakefs 
integration), so good to see it will be part of 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...@arrow.apache.org

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

Reply via email to