dramaticlly commented on issue #10535: URL: https://github.com/apache/iceberg/issues/10535#issuecomment-2177054658
thank you Szehon, I think `DeleteFiles` API only work with data files, delete files but not for manifest files. I tried to leverage delete by file path in [DeleteFiles](https://github.com/apache/iceberg/blob/main/api/src/main/java/org/apache/iceberg/DeleteFiles.java#L45) ->[MergingSnapshotProducer](https://github.com/apache/iceberg/blob/main/core/src/main/java/org/apache/iceberg/MergingSnapshotProducer.java#L187-L202) but it does not work. I agreed that deleteManifest sounds like a right way to do this instead of stub in rewriteManifests. I am aware of the fix 8599 above, it's just that some damage has occurred this time and we are looking forward to a way to fix corrupted table. There can be potentially other reasons for dropping a manifest from iceberg metadata, which is not relate to 8599. -- 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
