You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "Pradeep Garigipati (Jira)" <ji...@apache.org> on 2022/03/02 18:13:00 UTC

[jira] [Commented] (ARROW-14134) [C++][Compute] Standardize generator dispatchers

    [ https://issues.apache.org/jira/browse/ARROW-14134?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17500322#comment-17500322 ] 

Pradeep Garigipati commented on ARROW-14134:
--------------------------------------------

After brain storming (with [~edponce]) for a bit on generator dispatchers in [codegen_internal.h|https://github.com/apache/arrow/blob/master/cpp/src/arrow/compute/kernels/codegen_internal.h#L988] source file, we have created a [google document|https://docs.google.com/document/d/1xIwOGSsQiykGvmX1bK3iRWDbJagGeAC0-4rH1xgVMBI/edit?usp=sharing] outlining a proposal to refactor the names of generator dispatchers to follow a systematic pattern. 

We are sharing the document here to take the discussion further with community feedback.

Thank you.

[~lidavidm] 

> [C++][Compute] Standardize generator dispatchers
> ------------------------------------------------
>
>                 Key: ARROW-14134
>                 URL: https://issues.apache.org/jira/browse/ARROW-14134
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Eduardo Ponce
>            Assignee: Eduardo Ponce
>            Priority: Trivial
>              Labels: kernel
>             Fix For: 8.0.0
>
>
> codegen_internal.h formally defines generator dispatchers and there are a few GDs defined in kernel source files. GDs should be categorized, commented, and renamed appropriately.
> Also, the documentation for GD should be updated since more capable GDs have been added.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)