You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "Ian Cook (Jira)" <ji...@apache.org> on 2021/09/22 21:41:00 UTC
[jira] [Comment Edited] (ARROW-14085) [R] Expose null placement
option through sort bindings
[ https://issues.apache.org/jira/browse/ARROW-14085?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17418841#comment-17418841 ]
Ian Cook edited comment on ARROW-14085 at 9/22/21, 9:40 PM:
------------------------------------------------------------
The behavior before this (and the default behavior after it) is that {{NaN}} and {{NA}} values come _after_ other values. This is consistent with what {{dplyr::arrange()}} does, and dplyr offers no option to change this behavior, so I don't think we need to change our {{arrange()}} binding at all.
We do need to add this option to the {{sort_indices}} and {{array_sort_indices}} bindings.
was (Author: icook):
The behavior before this (and the default behavior after it) is that {{NaN}} and {{NA}} values come _after_ other values. This is consistent with what {{dplyr::arrange()}} does, and dplyr offers no option to change this behavior, so I don't think we need to change our {{arrange()}} binding at all.
> [R] Expose null placement option through sort bindings
> ------------------------------------------------------
>
> Key: ARROW-14085
> URL: https://issues.apache.org/jira/browse/ARROW-14085
> Project: Apache Arrow
> Issue Type: Improvement
> Components: R
> Reporter: Ian Cook
> Assignee: Ian Cook
> Priority: Major
> Labels: kernel
> Fix For: 6.0.0
>
>
> ARROW-12063 added a null placement option to the sort kernels and to {{OrderBySinkNode}} in the C++ library. Expose this through the R bindings.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)