Re: [PR] Move puffin crate contents inside iceberg crate [iceberg-rust]

2024-12-13 Thread via GitHub
Xuanwo commented on PR #789: URL: https://github.com/apache/iceberg-rust/pull/789#issuecomment-2540906883 Thank you, @Fokko, for the feedback. Since neither of us has a strong opinion on this and the suggestion comes from @liurenjie1024 with agreement from @fqaiser94, I believe there’s no n

Re: [PR] Move puffin crate contents inside iceberg crate [iceberg-rust]

2024-12-12 Thread via GitHub
Fokko commented on PR #789: URL: https://github.com/apache/iceberg-rust/pull/789#issuecomment-2540805212 I can see value in both. It could be that folks just want to have logic to read the Puffin files, and then they could just use the crate (for example PyIceberg) without having to pull in

[PR] Move puffin crate contents inside iceberg crate [iceberg-rust]

2024-12-12 Thread via GitHub
fqaiser94 opened a new pull request, #789: URL: https://github.com/apache/iceberg-rust/pull/789 Part of https://github.com/apache/iceberg-rust/issues/744 # Summary - Move contents of the puffin crate over to the existing iceberg crate - Delete the puffin crate # Context