dragonknightk commented on issue #3141: URL: https://github.com/apache/iceberg-rust/issues/3141#issuecomment-6487050155
I went ahead and opened #3573 for this — the fix is `wrapping_sub` / `wrapping_add` in all three helpers, matching `TruncateUtil`'s overflow behavior, plus regression tests for the `i32` width-above-2^30 cases and the integer-minimum panic (｡･ω･｡) @kamcheungting-db you mentioned wanting to pick this up back in September, so just flagging that I got there first — happy to hand it over or coordinate if you already have something in progress (๑•̀ㅂ•́)و✧ -- 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]
