DerGut commented on PR #1368:
URL: https://github.com/apache/iceberg-rust/pull/1368#issuecomment-2931632298

   ## 🏁 My plan to get this PR to the finish line
   
   - **abandon Azurite support** (for now): Unfortunately this sacrifices 
integration tests and easy local development. But for the latter at least, 
users can still iterate with a different FileIO implementation
   - **test it**: I'm planning some file operations against a real ADLS account
   
   ## 🙏 And what I need from the community/ reviewers
   - [ ] **Path format**: Do we want to settle on the long form, the short one 
or both?
   - [ ] **Defensive/ typed vs. concise/ dynamic-ish code**: I've defaulted to 
a more defensive programming approach when implementing the path+configuration 
parsing+matching logic. This also helped me to understand what I was working 
with. To be fair, this bloats the code and if preferred we can still simplify 
it.


-- 
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: issues-unsubscr...@iceberg.apache.org

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


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

Reply via email to