Xuanwo commented on PR #1755: URL: https://github.com/apache/iceberg-rust/pull/1755#issuecomment-3479804046
Hi, @liurenjie1024 and @CTTY, I have thinked about this again. I think we can split the concept around `fileio` and `fileio-provider` (or just using the words in this PR, `Storage` and `Fileio`). As discussed in https://github.com/apache/iceberg-rust/issues/1797, sometimes people don't care about FileIO (the thing in java) at all. They will initiate, build and manage their own IO abstraction and only want to be used as `Storage` in the iceberg-rust. So, I think we should make `FileIO` optional in the future and only depends on `Storage`. However it can be tricky since we do need the power to build a storage from `s3://bucket/name`. We can discuss them face to face and come up with a proposal. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
