Re: [PR] feat: Extract FileRead and FileWrite trait [iceberg-rust]

2024-05-14 Thread via GitHub
liurenjie1024 merged PR #364: URL: https://github.com/apache/iceberg-rust/pull/364 -- 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...@ic

Re: [PR] feat: Extract FileRead and FileWrite trait [iceberg-rust]

2024-05-10 Thread via GitHub
Xuanwo commented on code in PR #364: URL: https://github.com/apache/iceberg-rust/pull/364#discussion_r1597332691 ## crates/iceberg/src/io.rs: ## @@ -206,6 +205,35 @@ impl FileIO { } } +/// The struct the represents the metadata of a file. +/// +/// TODO: we can add last

Re: [PR] feat: Extract FileRead and FileWrite trait [iceberg-rust]

2024-05-10 Thread via GitHub
sdd commented on code in PR #364: URL: https://github.com/apache/iceberg-rust/pull/364#discussion_r1597146716 ## crates/iceberg/src/io.rs: ## @@ -206,6 +205,35 @@ impl FileIO { } } +/// The struct the represents the metadata of a file. +/// +/// TODO: we can add last mod

Re: [PR] feat: Extract FileRead and FileWrite trait [iceberg-rust]

2024-05-10 Thread via GitHub
sdd commented on code in PR #364: URL: https://github.com/apache/iceberg-rust/pull/364#discussion_r1597145954 ## crates/iceberg/src/arrow/reader.rs: ## @@ -187,3 +197,43 @@ impl ArrowReader { } } } + +/// ArrowFileReader is a wrapper around a FileRead that impls p

Re: [PR] feat: Extract FileRead and FileWrite trait [iceberg-rust]

2024-05-10 Thread via GitHub
Xuanwo commented on code in PR #364: URL: https://github.com/apache/iceberg-rust/pull/364#discussion_r1596407734 ## crates/iceberg/src/arrow/reader.rs: ## @@ -91,12 +98,15 @@ impl ArrowReader { Ok(try_stream! { while let Some(Ok(task)) = tasks.next().awai

Re: [PR] feat: Extract FileRead and FileWrite trait [iceberg-rust]

2024-05-10 Thread via GitHub
Xuanwo commented on code in PR #364: URL: https://github.com/apache/iceberg-rust/pull/364#discussion_r1596405143 ## crates/iceberg/src/io.rs: ## @@ -206,6 +205,35 @@ impl FileIO { } } +/// The struct the represents the metadata of a file. +/// +/// TODO: we can add last

Re: [PR] feat: Extract FileRead and FileWrite trait [iceberg-rust]

2024-05-10 Thread via GitHub
Xuanwo commented on code in PR #364: URL: https://github.com/apache/iceberg-rust/pull/364#discussion_r1596403546 ## crates/iceberg/src/arrow/reader.rs: ## @@ -187,3 +197,43 @@ impl ArrowReader { } } } + +/// ArrowFileReader is a wrapper around a FileRead that impl

Re: [PR] feat: Extract FileRead and FileWrite trait [iceberg-rust]

2024-05-09 Thread via GitHub
sdd commented on code in PR #364: URL: https://github.com/apache/iceberg-rust/pull/364#discussion_r1596342297 ## crates/iceberg/src/io.rs: ## @@ -206,6 +205,35 @@ impl FileIO { } } +/// The struct the represents the metadata of a file. +/// +/// TODO: we can add last mod

Re: [PR] feat: Extract FileRead and FileWrite trait [iceberg-rust]

2024-05-07 Thread via GitHub
Xuanwo commented on PR #364: URL: https://github.com/apache/iceberg-rust/pull/364#issuecomment-2098851128 cc @Fokko @liurenjie1024 @ZENOTME for reviews. -- 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