Xuanwo commented on code in PR #802:
URL: https://github.com/apache/iceberg-rust/pull/802#discussion_r1887055534

##########
crates/iceberg/src/io/file_io.rs:
##########
@@ -165,7 +175,7 @@ impl FileIOBuilder {
     /// Fetch the scheme string.
     ///
     /// The scheme_str will be empty if it's None.
-    pub(crate) fn into_parts(self) -> (String, HashMap<String, String>) {
+    pub fn into_parts(self) -> (String, HashMap<String, String>) {

Review Comment:
   > What's the goal of the `scheme_str`? If we want to have a specific impl, 
we could also pass that into the configuration itself. This is of course bound 
to a language, [for example 
`py-io-impl`](https://py.iceberg.apache.org/configuration/#fileio) and in Java 
it is just `io-impl`.
   
   Makes sense to me.
   
   I exposed the necessary elements as they are to build `FileIO` for now. I 
can address this in another PR that removes the requirement for `scheme` to 
build `FileIO`.



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