etseidl opened a new pull request, #9997: URL: https://github.com/apache/arrow-rs/pull/9997
# Which issue does this PR close? - Closes #9995. # Rationale for this change See issue. Improve code maintainability by using thrift macro to generate `LogicalType` serialization code. # What changes are included in this PR? Adds a new macro to generate code for a Thrift `union` that needs to be forward compatible. Does this by adding a catchall `_Unknown` variant for unknown field ids. # Are there any user-facing changes? Yes this is a breaking API change because the `LogicalType` enum will now use tuple variants rather than struct. This also makes public some structs that were previously private. -- 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]
