buraksenn opened a new pull request, #24726: URL: https://github.com/apache/datafusion/pull/24726
## Which issue does this PR close? - Closes #24170. - Follow-up to #24483. ## Rationale for this change #24483 added checked integer conversions across logical and physical plan serialization. Common protobuf conversions for file-format options and constraints still used unchecked casts, allowing oversized values to silently truncate on 32-bit targets. ## What changes are included in this PR? - Apply the checked conversion helper from #24483 to common options and constraints. - Make affected conversions fallible, propagating errors instead of truncating or panicking. - Update conversion coverage and the 56.0.0 upgrade guide. The protobuf wire format is unchanged. ## Are these changes tested? Yes. Existing option round-trip tests pass, and tests cover the fallible public conversion contracts and malformed constraints. ## Are there any user-facing changes? Oversized protobuf values now return an error instead of silently truncating. Some public `From` conversions are now `TryFrom`, so downstream callers must handle the resulting error. This migration is documented in the 56.0.0 upgrade guide. -- 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]
