[ARROW-12749] [C++] Unnecessary copy cause by constructing RecordBatch/Table/Schema from lvalues

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 5.0.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/28491
• Assignee: yibocai#1
• Reporter: yibocai#1
• Created: 12/May/21 09:50
• Updated: 11/Jan/23 08:28
• Resolved: 12/May/21 11:20

Description

Noticed some codes don't move lvalue augment when constructing RecordBatch/Table/Schema. It may cause unnecessary data copying, especially std::vector<std::shared_ptr> which can be big.

Issue Links

• links to: GitHub Pull Request #10301