[ARROW-8727] [C++] Do not require struct-initialization of StringConverter<T> to parse strings to other types

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.0.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/24880
• Assignee: Wes McKinney
• Reporter: Wes McKinney
• Created: 07/May/20 02:43
• Updated: 11/Jan/23 08:02
• Resolved: 08/May/20 20:24

Description

I ran into this issue while working on refactoring kernels. StringConverter<T> must be initialized to be able to support parametric types like Timestamp, but this produces an awkwardness and possibly a performance penalty (I haven't measured yet) in inlined functions.

In any case, I'm refactoring everything to be static non-stateful

Issue Links

• links to: GitHub Pull Request #7120