SEZ9 commented on issue #12612: URL: https://github.com/apache/seatunnel/issues/12612#issuecomment-5976298691
Thanks for the clear report with reproduction values. This is a well-scoped follow-up to #12605; a PR applying the same per-family range check for the BIGINT/LONG branch (BigDecimal/BigInteger vs Long.MIN_VALUE..MAX_VALUE, reject NaN/Infinity for Double/Float, pass exact integral wrappers through) would be welcome. Please include unit tests covering the four inputs listed (2^64+5, 2^64, NaN, 1e30) for both CAST (expects failure) and TRY_CAST (expects NULL), and confirm that existing in-range BigDecimal -> BIGINT casts still succeed. Consider whether the COALESCE/IFNULL result-type inference (first non-null argument rather than widest type) should be tracked as a separate issue, since it is a distinct behavior from the CAST narrowing fix. <!-- streview-comment:1514 --> -- 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]
