minor fixes (#564)

Co-authored-by: Kevin Liu <kevinjqliu@users.noreply.github.com>

Author: HonahX

pyiceberg/table/__init__.py

@@ -1056,8 +1056,9 @@ def append(self, df: pa.Table) -> None:
 
         _check_schema_compatible(self.schema(), other_schema=df.schema)
         # cast if the two schemas are compatible but not equal
-        if schema_to_pyarrow(self.schema()) != df.schema:
-            df = df.cast(schema_to_pyarrow(self.schema()))
+        table_arrow_schema = schema_to_pyarrow(self.schema())
+        if table_arrow_schema != df.schema:
+            df = df.cast(table_arrow_schema)
 
         merge = _MergingSnapshotProducer(operation=Operation.APPEND, table=self)
 
@@ -1096,8 +1097,9 @@ def overwrite(self, df: pa.Table, overwrite_filter: BooleanExpression = ALWAYS_T
 
         _check_schema_compatible(self.schema(), other_schema=df.schema)
         # cast if the two schemas are compatible but not equal
-        if schema_to_pyarrow(self.schema()) != df.schema:
-            df = df.cast(schema_to_pyarrow(self.schema()))
+        table_arrow_schema = schema_to_pyarrow(self.schema())
+        if table_arrow_schema != df.schema:
+            df = df.cast(table_arrow_schema)
 
         merge = _MergingSnapshotProducer(
             operation=Operation.OVERWRITE if self.current_snapshot() is not None else Operation.APPEND,

tests/integration/test_reads.py

@@ -230,7 +230,6 @@ def test_ray_nan_rewritten(catalog: Catalog) -> None:
 def test_ray_not_nan_count(catalog: Catalog) -> None:
     table_test_null_nan_rewritten = catalog.load_table("default.test_null_nan_rewritten")
     ray_dataset = table_test_null_nan_rewritten.scan(row_filter=NotNaN("col_numeric"), selected_fields=("idx",)).to_ray()
-    print(ray_dataset.take())
     assert ray_dataset.count() == 2
 
 

tests/integration/test_writes.py

@@ -481,7 +481,6 @@ def test_write_parquet_other_properties(
     properties: Dict[str, Any],
     expected_kwargs: Dict[str, Any],
 ) -> None:
-    print(type(mocker))
     identifier = "default.test_write_parquet_other_properties"
 
     # The properties we test cannot be checked on the resulting Parquet file, so we spy on the ParquetWriter call instead