You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "Antoine Pitrou (Jira)" <ji...@apache.org> on 2021/06/09 18:57:00 UTC

[jira] [Resolved] (ARROW-12951) [C++] Refactor StringTransform

     [ https://issues.apache.org/jira/browse/ARROW-12951?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Antoine Pitrou resolved ARROW-12951.
------------------------------------
    Fix Version/s: 5.0.0
       Resolution: Fixed

Issue resolved by pull request 10496
[https://github.com/apache/arrow/pull/10496]

> [C++] Refactor StringTransform
> ------------------------------
>
>                 Key: ARROW-12951
>                 URL: https://issues.apache.org/jira/browse/ARROW-12951
>             Project: Apache Arrow
>          Issue Type: Task
>          Components: C++
>            Reporter: Antoine Pitrou
>            Assignee: Antoine Pitrou
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 5.0.0
>
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> In {{compute/kernels/scalar_string.cc}, currently there is a non-trivial inheritance relationship between {{StringTransform}} and its derived implementation classes. Also, the derived classes have to be templated on the actual string type even though their code doesn't care, making generated code size probably bigger than it should be.
> Instead, there could be a {{StringTransformExec}} class that delegates to an independent class for per-kernel specifics (mostly: the {{MaxCodeunits}} and {{Transform}} methods).



--
This message was sent by Atlassian Jira
(v8.3.4#803005)