Update mehtod signature of _files()

Author: soumya-ghosh

pyiceberg/table/__init__.py

@@ -4308,7 +4308,9 @@ def history(self) -> "pa.Table":
 
         return pa.Table.from_pylist(history, schema=history_schema)
 
-    def _files(self, snapshot_id: Optional[int] = None, file_content_type: str = "all") -> "pa.Table":
+    def _files(
+        self, snapshot_id: Optional[int] = None, data_file_content_types: Optional[List[DataFileContent]] = None
+    ) -> "pa.Table":
         import pyarrow as pa
 
         from pyiceberg.io.pyarrow import schema_to_pyarrow
@@ -4365,9 +4367,7 @@ def _readable_metrics_struct(bound_type: PrimitiveType) -> pa.StructType:
         for manifest_list in snapshot.manifests(io):
             for manifest_entry in manifest_list.fetch_manifest_entry(io):
                 data_file = manifest_entry.data_file
-                if file_content_type == "data" and data_file.content != DataFileContent.DATA:
-                    continue
-                if file_content_type == "delete" and data_file.content == DataFileContent.DATA:
+                if data_file_content_types and data_file.content not in data_file_content_types:
                     continue
                 column_sizes = data_file.column_sizes or {}
                 value_counts = data_file.value_counts or {}
@@ -4419,10 +4419,10 @@ def files(self, snapshot_id: Optional[int] = None) -> "pa.Table":
         return self._files(snapshot_id)
 
     def data_files(self, snapshot_id: Optional[int] = None) -> "pa.Table":
-        return self._files(snapshot_id, "data")
+        return self._files(snapshot_id, [DataFileContent.DATA])
 
     def delete_files(self, snapshot_id: Optional[int] = None) -> "pa.Table":
-        return self._files(snapshot_id, "delete")
+        return self._files(snapshot_id, [DataFileContent.POSITION_DELETES, DataFileContent.EQUALITY_DELETES])
 
 
 class _ManifestMergeManager(Generic[U]):