jder commented on issue #409:
URL: 
https://github.com/apache/arrow-rs-object-store/issues/409#issuecomment-2988909607

   Ok, thanks. One last thought: if AmazonS3Builder::build returned (Self, 
Path) and ObjectStoreScheme::parse returned Self (and not Path), it would let 
us remove the path/bucket logic from ObjectStore::parse which is mostly 
duplicated inside AmazonS3Builder::parse_url, and also make with_url populate 
that path as I originally expected. 
   
   What do you think? Is that too big of a change? 


-- 
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