[ARROW-5777] [C++] BasicDecimal128 is a small object it doesn't always make sense to pass by const ref

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.15.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/22199
• Assignee: Micah Kornfield
• Reporter: Micah Kornfield
• Created: 28/Jun/19 07:41
• Updated: 11/Jan/23 07:42
• Resolved: 08/Jul/19 18:38

Description

For the builder use case we saw 10% improvement in performance by not using const ref.
Add benchmarks and look at where it makes sense to avoid passing by ref.

Issue Links

• links to: GitHub Pull Request #4739