refactor deleted_data_files

Author: jqin61

Makefile

@@ -42,7 +42,7 @@ test-integration:
 	docker-compose -f dev/docker-compose-integration.yml up -d
 	sleep 5
 	docker-compose -f dev/docker-compose-integration.yml exec -T spark-iceberg ipython ./provision.py
-	poetry run pytest tests/ -v -m france ${PYTEST_ARGS} -s
+	poetry run pytest tests/ -v -m integration ${PYTEST_ARGS}
 
 
 test-integration-rebuild:

pyiceberg/expressions/__init__.py

@@ -664,13 +664,11 @@ class LiteralPredicate(UnboundPredicate[L], ABC):
     literal: Literal[L]
 
     def __init__(self, term: Union[str, UnboundTerm[Any]], literal: Union[L, Literal[L]]):  # pylint: disable=W0621
-        # print("lets check the type", literal, type(literal), term)
         super().__init__(term)
         self.literal = _to_literal(literal)  # pylint: disable=W0621
 
     def bind(self, schema: Schema, case_sensitive: bool = True) -> BoundLiteralPredicate[L]:
         bound_term = self.term.bind(schema, case_sensitive)
-        # print("last min", self.term, self.literal, bound_term, bound_term.ref().field, bound_term.ref().field.field_type)
         lit = self.literal.to(bound_term.ref().field.field_type)
 
         if isinstance(lit, AboveMax):

pyiceberg/expressions/visitors.py

@@ -490,7 +490,6 @@ def visit_not_null(self, term: BoundTerm[L]) -> bool:
         return term.eval(self.struct) is not None
 
     def visit_equal(self, term: BoundTerm[L], literal: Literal[L]) -> bool:
-        # print("adrian kernel is called", term, literal, "eval is, ", term.eval(self.struct), f"{self.struct=}")
         return term.eval(self.struct) == literal.value
 
     def visit_not_equal(self, term: BoundTerm[L], literal: Literal[L]) -> bool:
@@ -554,8 +553,6 @@ def __init__(self, partition_struct_schema: Schema, partition_filter: BooleanExp
         self.partition_filter = bind(partition_struct_schema, rewrite_not(partition_filter), case_sensitive)
 
     def eval(self, manifest: ManifestFile) -> bool:
-        # print(f"about why partition evalutes failed {manifest.partitions=}")
-        # print(f"about why partition evalutes failed {manifest.partitions[0].lower_bound=}, {manifest.partitions[0].upper_bound=}, {self.partition_filter=}")
         if partitions := manifest.partitions:
             self.partition_fields = partitions
             return visit(self.partition_filter, self)