Re: [I] Decouple building and serialization [iceberg-rust]

2025-01-25 Thread via GitHub
Sl1mb0 commented on issue #778: URL: https://github.com/apache/iceberg-rust/issues/778#issuecomment-2614009219 > A builder method for ManifestFile/ManifestList, instead of building them using writers only? Yes - I think that would be ideal. This would help distinguish between buildin

Re: [I] Decouple building and serialization [iceberg-rust]

2024-12-18 Thread via GitHub
liurenjie1024 commented on issue #778: URL: https://github.com/apache/iceberg-rust/issues/778#issuecomment-2552958412 Hi, @Sl1mb0 Thanks for raising this! If I understand correctly, you are asking to parts: 1. A builder method for `ManifestFile`/`ManifestList`, instead of building them u

Re: [I] Decouple building and serialization [iceberg-rust]

2024-12-13 Thread via GitHub
Sl1mb0 commented on issue #778: URL: https://github.com/apache/iceberg-rust/issues/778#issuecomment-2541752081 I should clarify: I would be happy do this work at some point but would like to know if it would be accepted. -- This is an automated message from the Apache Git Service. To resp

Re: [I] Decouple building and serialization [iceberg-rust]

2024-12-12 Thread via GitHub
Sl1mb0 commented on issue #778: URL: https://github.com/apache/iceberg-rust/issues/778#issuecomment-2539813154 > If I understand correctly if you could provide your own implementation of FileIO, would you be able to make it work? This would avoid the copy. Hmm - this may work, but it'

Re: [I] Decouple building and serialization [iceberg-rust]

2024-12-12 Thread via GitHub
Fokko commented on issue #778: URL: https://github.com/apache/iceberg-rust/issues/778#issuecomment-2538270065 Hey @Sl1mb0, Thanks for raising this issue. If I understand correctly if you could provide your own implementation of FileIO, would you be able to make it work? This would avoid the

[I] Decouple building and serialization [iceberg-rust]

2024-12-11 Thread via GitHub
Sl1mb0 opened a new issue, #778: URL: https://github.com/apache/iceberg-rust/issues/778 At the moment, the building and serialization of Iceberg metadata is coupled together. For example, let's say I want to build a `ManifestFile` that I then add to a `ManifestList`: (some cod