liurenjie1024 commented on issue #1533: URL: https://github.com/apache/iceberg-rust/issues/1533#issuecomment-3102124550
> > Do you have concrete use cases when this is useful? e.g. Updating snapshot properties without manipulating table data. > > In [moonlink](https://github.com/Mooncake-Labs/moonlink), we support compaction via self implemented catalog, which is possible to have only data files to delete, but nothing to add (which is not supported as of now). Thanks @dentiny for this suggestion. But your use case makes me feel that adding a `SetSnapshotProperty` tx action is a workaround for the missing of some other tx actions. I have a big concern about this approach and don't think this is the right direction. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
