felipecrv opened a new pull request, #36316:
URL: https://github.com/apache/arrow/pull/36316
### Rationale for this change
`CleanListOffsets` is taking output parameters when it could return multiple
buffers in a vector exactly how they are supposed to be used to construct the
list-like array.
Besides that, having the logic for setting the validity buffer split between
it and the caller is unnecessarily confusing.
### What changes are included in this PR?
- Addition of a new constructor for `MapArray` that can accept a
pre-allocated `BufferVector`
- Removal of output parameter in `CleanListOffsets` and change of return
type
### Are these changes tested?
Yes, by existing tests. A pre-existing `MapArray` ctor delegates to this new
ctor, so all tests that exercise the old ctor also exercise the new.
--
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]
