alamb opened a new pull request, #7626: URL: https://github.com/apache/arrow-rs/pull/7626
# Which issue does this PR close? - Follow on to https://github.com/apache/arrow-rs/pull/7625 from @Dandandan # Rationale for this change I want to eventually remove `gc_string_view` but currently the unit tests are in terms of that function # What changes are included in this PR? Rewrite tests to be in terms of `coalesce` instead Also, 1. Add additional coverage for the issue we saw in 2. Add add coverage for the case where there are data buffers in the view, but they are not referenced by any view https://github.com/apache/arrow-rs/pull/7625#discussion_r2134634467 Codecov of this module is now 100% # Are there any user-facing changes? If there are user-facing changes then we may require documentation to be updated before approving the PR. If there are any breaking changes to public APIs, please call them out. -- 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]
