bartash opened a new pull request, #9183:
URL: https://github.com/apache/iceberg/pull/9183

   When a snapshot is expired as part of a transaction, the snapshot file(s) 
should be deleted when the transaction commits. A recent change (#6634) ensured 
that files are not deleted when they have also been committed as part of a 
transaction, but this breaks the simple case where no new files are committed. 
Fix this by not skipping deletion when the list of committed files is empty.
   
   Closes #9182 
   
   TESTING:
   
   Extended a unit test to ensure that snapshot files are deleted. Ran the test 
without the fix on a branch where #6634 was reverted to show that this is a 
regression.


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