[ARROW-4399] [C++] Remove usage of "extern template class" from NumericArray<T>

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.13.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/20963
• Assignee: Wes McKinney
• Reporter: Wes McKinney
• Created: 28/Jan/19 04:24
• Updated: 11/Jan/23 07:33
• Resolved: 28/Jan/19 18:20

Description

We aren't using this to any benefit right now (only the simple constructor is being instantiated once, everything else in inline), and the rules about template visibility and linkage vary a lot between compilers, as evidenced by patches such as https://github.com/apache/arrow/pull/3503

In general I think we should try not to use "extern template class" at all in the codebase for the problems that come with it

Issue Links

• relates to: ARROW-4394 [C++] Can't build with debug option and MinGW
• links to: GitHub Pull Request #3509