[ARROW-3169] [C++] Break array-test.cc and array.cc into multiple compilation units

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.12.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/19516
• Assignee: Antoine Pitrou
• Reporter: Wes McKinney
• Created: 04/Sep/18 19:33
• Updated: 11/Jan/23 07:25
• Resolved: 29/Nov/18 15:40

Description

To improve readability I suggest splitting array-test.cc into multiple compilation units (which will still be linked into an executable array-test). We can do the same thing with array.h/array.cc, while maintaining the arrow/array.h public header. Some of these could go into arrow/columnar or arrow/impl, or something similar.

Issue Links

• supercedes: ARROW-3254 [C++] Add option to ADD_ARROW_TEST to compose a test executable from multiple .cc files containing unit tests
• links to: GitHub Pull Request #3047