[GitHub] [iceberg] RussellSpitzer commented on pull request #2810: API: Use delete instead of remove in actions for clarity

[GitBox <gi...@apache.org>]

RussellSpitzer commented on pull request #2810:
URL: https://github.com/apache/iceberg/pull/2810#issuecomment-878641339


   My only worry here is we have two senses of delete in the API. Logical deletes and physical deletes, before the names were distinct since "delete" apis only did logical deletes while "remove" apis actually removed files from your file system. I also only worry about this on about a 2 out of 10 worry scale so i'm +1 on moving forward if @aokolnychyi feels strongly about it :)


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org