[ARROW-3906] [C++] Break builder.cc into multiple compilation units

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.11.1
• Fix Version/s: 0.12.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/15888
• Assignee: Antoine Pitrou
• Reporter: Antoine Pitrou
• Created: 29/Nov/18 15:47
• Updated: 11/Jan/23 07:30
• Resolved: 04/Dec/18 01:19

Description

To improve readability I suggest splitting builder.cc into independent compilation units. Concrete builder classes are generally independent of each other. The only concern is whether inlining some of the base class implementations is important for performance.

This would also make incremental compilation faster when changing one of the concrete classes.

Issue Links

• links to: GitHub Pull Request #3076