Re: [DISCUSS] API: Rename RowDelta deleteFile to removeRows

2025-05-29 Thread Ryan Blue
+1 It is good to have consistency within the RowDelta API. And I think it is a good idea in general to use "remove" to refer to removing a file from metadata, rather than "delete" because you can add or remove delete files. On Thu, May 29, 2025 at 11:46 AM Russell Spitzer wrote: > Ryan pointed

[DISCUSS] API: Rename RowDelta deleteFile to removeRows

2025-05-29 Thread Russell Spitzer
Ryan pointed out to me that whenI added the "deleteFile" method I was not following the convention already being used within the RowDelta operation and instead had copied the OverwriteFiles API. To fix this I think it would be great to change the API to "removeRows" to match the other APIs in the c