mxm commented on PR #12049:
URL: https://github.com/apache/iceberg/pull/12049#issuecomment-2609557685
> @mxm: Please remove the 1.18, 1.19 changes from the PR. It is much easier
to review this way, and apply changes required by the reviewer. When the PR has
been merged, we can backport the changes to the other Flink versions.
Makes sense! Done.
> QQ: What happens when we have a type discrepancy between the Iceberg type
and the RawData type? Could we have issues with other conversions? Do we have a
way to prevent those?
Type discrepancies between Iceberg and Flink types will error in Flink's
TypeSerializer for a given field. For example, an int field will use
IntSerializer which only accepts Integer. This will raise an NoSuchMethodError
during serialization. As long as we use the same serializer also for
deserialization, we should be fine. That is the case.
--
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
