perf(map): optimize MapRef to reduce more copy

[xxchan]

See #17690 (comment)

[xxchan]

Perhaps we should refactor ListRef to use Arc<ArrayImpl>.

It seems our array implementation has flaws to support such kind of optimizations. Should we switch to arrow-rs? 🤪

[wangrunji0408]

Should we switch to arrow-rs? 🤪

This is technically possible and I agree with it. But there will be endless migration work. 🤪

[BugenZhao]

It seems our array implementation has flaws to support such kind of optimizations. Should we switch to arrow-rs? 🤪

Why? 😕 Can't we make it a enum to support both cases? Just like the idea of Cow.

[github-actions]

This issue has been open for 60 days with no activity.

If you think it is still relevant today, and needs to be done in the near future, you can comment to update the status, or just manually remove the no-issue-activity label.

You can also confidently close this issue as not planned to keep our backlog clean.
Don't worry if you think the issue is still valuable to continue in the future.
It's searchable and can be reopened when it's time. 😄