[ARROW-3778] [C++] Don't put implementations in test-util.h

• 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/20132
• Assignee: Wes McKinney
• Reporter: Antoine Pitrou
• Created: 13/Nov/18 18:50
• Updated: 11/Jan/23 07:29
• Resolved: 22/Nov/18 01:07

Description

test-util.h is included in most (all?) test files, and it's quite long to compile because it includes many other files and recompiles helper functions all the time. Instead we should have only declarations in test-util.h and put implementations in a separate .cc file.

Issue Links

• links to: GitHub Pull Request #2997