rshkv commented on PR #863: URL: https://github.com/apache/iceberg-rust/pull/863#issuecomment-2577942180
I've rebased on #870 and #872 to address the follow: * The entries table now lives in a separate `entries.rs` file. * Batches for manifest files are now computed asynchronously. I haven't address @liurenjie1024's point about `schema()` returning an Iceberg schema instead of an Arrow one. We have issue #868 and PR #871 but I'm not sure that's what we want generally and whether you want this to happen before this PR. Otherwise this is ready for another review. -- 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
