amogh-jahagirdar commented on PR #11131:
URL: https://github.com/apache/iceberg/pull/11131#issuecomment-2517917326
   @pvary I'll double check that test but my suspicion is you're seeing the 
behavior from this particular change 
https://github.com/apache/iceberg/pull/11131#discussion_r1815710103, this has 
the rationale.
   
    My suspicion is that the test used to rely on the eager rewriting of 
manifests with only aged out deletes, and that's no longer the case.  If that's 
the case, what's happening is that we're not eagerly rewriting manifests which 
*only* have aged out deletes since it's not required for correctness and it's 
best to avoid any additional work and risk failure for things not related to 
metadata correctness. Important note is that, if a manifest needs to be 
rewritten for other purposes *and* has aged out deletes, the aged out deletes 
will be rewritten to reduce metadata storage. Let me take a deeper look though 
to confirm that's what happening for the test case 


-- 
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

Reply via email to