syun64 commented on code in PR #829:
URL: https://github.com/apache/iceberg-python/pull/829#discussion_r1676337956
##########
pyiceberg/table/__init__.py:
##########
@@ -484,10 +484,6 @@ def append(self, df: pa.Table, snapshot_properties:
Dict[str, str] = EMPTY_DICT)
_check_schema_compatible(
self._table.schema(), other_schema=df.schema,
downcast_ns_timestamp_to_us=downcast_ns_timestamp_to_us
)
- # cast if the two schemas are compatible but not equal
Review Comment:
> @syun64 I want to get your take on this part. Due to the timestamp change,
do you know if the `df` need to be casted? There are a couple of different
parts involved in the write path. In particular, we need to look at the table
schema, the df schema, and the df itself. As well as dealing with bin-packing
and other transformations.
I have a PR open to try to fix this behavior:
https://github.com/apache/iceberg-python/pull/910 I think it's almost ready to
merge 😄
--
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
