szehon-ho commented on issue #12554: URL: https://github.com/apache/iceberg/issues/12554#issuecomment-2760322644
Yea I see, I guess its because the filePath will be different size in the rewritten delete file than the original delete file, so delete file size is wrong.. Originally I had it so we write the delete file first and then delete manifest, but we changed it to make the writing delete file parallel. I already fixed the upper and lower bounds for delete manifest, but the size is something I didnt fix (and is harder). We might need a large refactor to fix this (write the delete file first and then delete manifest). @flyrain any thoughts? -- 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
