alamb opened a new pull request, #5619: URL: https://github.com/apache/arrow-rs/pull/5619
Draft PR as I would love to know if anyone has any comments about this idea / approach # Which issue does this PR close? Part of #4585 # Rationale for this change While reviewing https://github.com/apache/arrow-rs/pull/5557 from @ariesdevil -- it seemed like the `12` byte view length was getting hard coded many places and the same code is repeated over and over I wanted to see if I could avoid that. # What changes are included in this PR? 1. Add a `InlineView` to mirror `BytesView` 2. Add a `View` enum that captures which view is used 3. Rewrite some of the code to use the new `View` # Are there any user-facing changes? Several new types -- 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]
