arnaudbriche commented on PR #356: URL: https://github.com/apache/iceberg-go/pull/356#issuecomment-2750755092
Hi @zeroshade , I confirm your latest commit fixed the issue in my tests. I think it's also important to tests scenarios where some of the removed datafiles are part of a manifest containing non-removed files. Test is nearly same as above, but files are not appended 1 by 1. I tested with this sequence: 1. append [data.parquet, data.1.parquet] 2. append [data.2.parquet, data.3.parquet] 3. append [data.4. parquet] 4. replace [data.4.parquet=data.1.parquet+data.2.parquet] And everything works as expected ! -- 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
