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

   # PR Description
   
   This is a rebased version of https://github.com/apache/iceberg/pull/15061 by 
@Heatao to resolve merge conflicts with the latest `main` branch.
   
   ## Summary
   
   DELETE operations did not respect custom snapshot properties set via session 
config (`spark.sql.iceberg.snapshot-property.*`), while INSERT and UPDATE 
operations did. This change adds `SparkWriteConf.extraSnapshotMetadata()` to 
the `deleteWhere()` method in `SparkTable` so that metadata-only DELETEs also 
pick up those properties.
   
   Changes (identical for Spark 4.0 and 4.1):
   - Apply `extraSnapshotMetadata` from `SparkWriteConf` in 
`SparkTable.deleteWhere()`
   - Add test coverage for copy-on-write, merge-on-read, and metadata-only 
DELETE operations
   
   Closes #15060.
   
   ## Context
   
   The original PR was approved by @amogh-jahagirdar, @ebyhr, and @yingjianwu98 
but could not be merged due to conflicts. The original author has been inactive 
for approximately two months. All original review feedback was addressed. I 
have only resolved the merge conflicts — the code changes are @Heatao's work.
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to