GeetKrishna commented on PR #1617:
URL: https://github.com/apache/iceberg-rust/pull/1617#issuecomment-3208540482
@emkornfield Right, with the current approach, it has potential for silent
data corruption because of Arrow's doc field dependency. @CTTY Thanks for the
references, I will use this for casting.
My updated approach uses safe bit-width casting for unsigned integer types
following the proven iceberg-python implementation
• **uint8/uint16 → int32**: Safe upcast with no overflow risk
• **uint32 → int64**: Safe upcast preserving full uint32 range
• **uint64 → explicit block**: Rather than risk data loss through unsafe
conversion, provide clear error guidance directing users to choose between
int64 (with range validation) or decimal (with full precision) based on their
specific requirements
Let me know if there are any concerns, I will have the changes out otherwise.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
