-
Notifications
You must be signed in to change notification settings - Fork 818
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deprecate MutableArrayData #2832
Comments
Hm, except for |
The hope was that all use-cases would be covered by a combination of the |
The most close one is |
I think I have made peace with the existence of MutableArrayData |
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
MutableArrayData
provides an API to copy chunks of data from source arrays into an output array, it is extremely flexible, however, its performance leaves a lot to be desired. In particular its reliance on dynamic dispatch per-run not only incurs dispatch overheads, but prevents the compiler from properly vectorising the code. There are also a number of other issues relating to capacity allocation (#347), null mask handling (#1230), dictionary handling (#506)There has been a fairly concerted effort over the last year or so to move away from use of MutableArrayData:
Describe the solution you'd like
I would like to eventually deprecate and remove MutableArrayData. I think the only major blockers to this are:
take
kernel that works across multipleRecordBatch
es #1523)Describe alternatives you've considered
Additional context
This can be viewed through the lens of porting @jorgecarleitao 's ideas from arrow2 and bringing them into arrow-rs
The text was updated successfully, but these errors were encountered: