robert3005 opened a new pull request, #10328: URL: https://github.com/apache/arrow-rs/pull/10328
# Which issue does this PR close? mailing list thread "Spec clarification on field names" https://lists.apache.org/thread/v2fvqjtfrvsrbm133yc7yjrxvjh0x3l5 # Rationale for this change Arrow supports names that are not valid c string names. Instead of having the caller validate themselves the ArrowSchema C FFI already validates on the rust side, instead of panicing propagate the error to the user # What changes are included in this PR? Propagate Cstring::new error to the user instead of panicing # Are these changes tested? Added tests # Are there any user-facing changes? Error instead of panic -- 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]
