HonahX commented on code in PR #569: URL: https://github.com/apache/iceberg-python/pull/569#discussion_r1669726783
########## pyiceberg/table/__init__.py: ########## @@ -539,6 +578,81 @@ def overwrite( for data_file in data_files: update_snapshot.append_data_file(data_file) + def delete(self, delete_filter: Union[str, BooleanExpression], snapshot_properties: Dict[str, str] = EMPTY_DICT) -> None: + """ + Shorthand for deleting record from a table. + Review Comment: One last minor comment: Shall we also mention that `delete` may produce different types of snapshot operation based on ``` - DELETE: In case existing Parquet files can be dropped completely. - REPLACE: In case existing Parquet files need to be rewritten ``` like the docstring of `overwrite`? -- 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