gstvg opened a new pull request, #6781: URL: https://github.com/apache/arrow-rs/pull/6781
# Which issue does this PR close? Follow-up on #6303 # Rationale for this change As noted by @wiedld on https://github.com/apache/arrow-rs/pull/6303#discussion_r1783545074, the existing tests coverage is quite fragile relative to the target architecture and enabled CPU features, leaving most logic untested on ARM, for example. # What changes are included in this PR? Instead of calling only the public `Array::logical_nulls`, tests of specific strategies also calls the correspondent private method directly. To do so, part of `logical_nulls` method, which builds the arguments to be passed to specific implementations, is made into a new private method `UnionArray::fields_logical_nulls`, and then used by both `logical_nulls` and the tests. # Are there any user-facing changes? No, just tests -- 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]
