Re: [PR] feat(core): expose remove_snapshots at Transaction API [iceberg-rust]

2025-03-11 Thread via GitHub
ZENOTME commented on PR #884: URL: https://github.com/apache/iceberg-rust/pull/884#issuecomment-2716708182 > > This is a hard limit in the server implementation. I am not sure about the reason for this design, as I am not familiar enough with the specification of iceberg. > > +1, I a

Re: [PR] feat(core): expose remove_snapshots at Transaction API [iceberg-rust]

2025-03-11 Thread via GitHub
Li0k commented on PR #884: URL: https://github.com/apache/iceberg-rust/pull/884#issuecomment-2716700631 > This is a hard limit in the server implementation. I am not sure about the reason for this design, as I am not familiar enough with the specification of iceberg. +1, I also encou

Re: [PR] feat(core): expose remove_snapshots at Transaction API [iceberg-rust]

2025-02-26 Thread via GitHub
ZENOTME commented on PR #884: URL: https://github.com/apache/iceberg-rust/pull/884#issuecomment-2684472066 > I am not sure if this is a server issue or a client issue, more research is needed. Looks like it depends on the catalog. For list of snapshots remove, it will apply to metada

Re: [PR] feat(core): expose remove_snapshots at Transaction API [iceberg-rust]

2025-01-12 Thread via GitHub
TennyZhuang commented on PR #884: URL: https://github.com/apache/iceberg-rust/pull/884#issuecomment-2586030157 https://github.com/apache/iceberg/blob/c7910bb401f7f7fd09010bede0d80f5d2164afd5/core/src/main/java/org/apache/iceberg/MetadataUpdateParser.java#L547 This is a hard limit in t

Re: [PR] feat(core): expose remove_snapshots at Transaction API [iceberg-rust]

2025-01-12 Thread via GitHub
TennyZhuang commented on PR #884: URL: https://github.com/apache/iceberg-rust/pull/884#issuecomment-2586026257 I am not sure if this is a server issue or a client issue, more research is needed. If only one snapshot_id is passed, it can be successful. ``` table: TableIdent