[jira] [Updated] (ARROW-6232) [C++] Rename Argsort kernel to SortToIndices

["Sutou Kouhei (Jira)" <ji...@apache.org>]

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

Sutou Kouhei updated ARROW-6232:
--------------------------------
    Summary: [C++] Rename Argsort kernel to SortToIndices  (was: [C++] Rename Argsort kernel to SortIndices)

> [C++] Rename Argsort kernel to SortToIndices
> --------------------------------------------
>
>                 Key: ARROW-6232
>                 URL: https://issues.apache.org/jira/browse/ARROW-6232
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Sutou Kouhei
>            Assignee: Sutou Kouhei
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 3h
>  Remaining Estimate: 0h
>
> "Argsort" is NumPy specific name. Other languages/libraries use
> different name:
>   * R: order
>     * https://cran.r-project.org/doc/manuals/r-release/fullrefman.pdf#Rfn.order
>   * MATLAB: sort
>     * https://mathworks.com/help/matlab/ref/sort.html
>     * "sort" returns sorted array and indices to sort array
>   * Julia: sortperm
>     * https://pkg.julialang.org/docs/julia/THl1k/1.1.1/base/sort.html#Base.sortperm
> It's better that we use general name because Arrow C++ isn't a NumPy
> compatible library.
> "SortIndices" means "sort that returns indices array". We can add
> "SortValues" or something for sort kernel that returns values array.
> "SortIndices" may be easily mistaken for "sort by indices".



--
This message was sent by Atlassian Jira
(v8.3.2#803003)