[jira] [Commented] (ARROW-15224) [R] Add binding for not_between() ternary kernel

2022-04-22 Thread Jira


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

Dragoș Moldovan-Grünfeld commented on ARROW-15224:
--

I will _close_ the issue with _won't fix._ Thanks __ 

> [R] Add binding for not_between() ternary kernel
> 
>
> Key: ARROW-15224
> URL: https://issues.apache.org/jira/browse/ARROW-15224
> Project: Apache Arrow
>  Issue Type: New Feature
>  Components: R
>Reporter: Eduardo Ponce
>Assignee: Dragoș Moldovan-Grünfeld
>Priority: Major
> Fix For: 9.0.0
>
>
> Add R binding for {{not_between()}} compute function from ARROW-15223.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)


[jira] [Commented] (ARROW-15224) [R] Add binding for not_between() ternary kernel

2022-04-22 Thread Eduardo Ponce (Jira)


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

Eduardo Ponce commented on ARROW-15224:
---

Based on these observations, it seems we conclude that a {{not_between}} 
function will not be included. So we can close this issue.

> [R] Add binding for not_between() ternary kernel
> 
>
> Key: ARROW-15224
> URL: https://issues.apache.org/jira/browse/ARROW-15224
> Project: Apache Arrow
>  Issue Type: New Feature
>  Components: R
>Reporter: Eduardo Ponce
>Assignee: Dragoș Moldovan-Grünfeld
>Priority: Major
> Fix For: 9.0.0
>
>
> Add R binding for {{not_between()}} compute function from ARROW-15223.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)


[jira] [Commented] (ARROW-15224) [R] Add binding for not_between() ternary kernel

2022-04-22 Thread Jira


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

Dragoș Moldovan-Grünfeld commented on ARROW-15224:
--

I think the situation is similar in {{dplyr}} - the data manipulation R package 
we link to. 

{code:r}
library(dplyr, warn.conflicts = FALSE)

starwars %>% 
  filter(between(height, 100, 150))
#> # A tibble: 5 × 14
#>   name  height  mass hair_color skin_color eye_color birth_year sex   
gender
#>   
 
#> 1 Leia Org…15049 brown  light  brown 19 fema… 
femin…
#> 2 Mon Moth…150NA auburn fair   blue  48 fema… 
femin…
#> 3 Watto137NA black  blue, grey yellowNA male  
mascu…
#> 4 Sebulba  11240 none   grey, red  orangeNA male  
mascu…
#> 5 Gasgano  122NA none   white, bl… black NA male  
mascu…
#> # … with 5 more variables: homeworld , species , films ,
#> #   vehicles , starships 

starwars %>% 
  filter(!between(height, 100, 150))
#> # A tibble: 76 × 14
#>name height  mass hair_color skin_color eye_color birth_year sex   
gender
#>   
 
#>  1 Luke Sk…17277 blond  fair   blue19   male  
mascu…
#>  2 C-3PO   16775gold   yellow 112   none  
mascu…
#>  3 R2-D29632white, bl… red 33   none  
mascu…
#>  4 Darth V…202   136 none   white  yellow  41.9 male  
mascu…
#>  5 Owen La…178   120 brown, gr… light  blue52   male  
mascu…
#>  6 Beru Wh…16575 brown  light  blue47   fema… 
femin…
#>  7 R5-D49732white, red red NA   none  
mascu…
#>  8 Biggs D…18384 black  light  brown   24   male  
mascu…
#>  9 Obi-Wan…18277 auburn, w… fair   blue-gray   57   male  
mascu…
#> 10 Anakin …18884 blond  fair   blue41.9 male  
mascu…
#> # … with 66 more rows, and 5 more variables: homeworld , species ,
#> #   films , vehicles , starships 
{code}

> [R] Add binding for not_between() ternary kernel
> 
>
> Key: ARROW-15224
> URL: https://issues.apache.org/jira/browse/ARROW-15224
> Project: Apache Arrow
>  Issue Type: New Feature
>  Components: R
>Reporter: Eduardo Ponce
>Assignee: Dragoș Moldovan-Grünfeld
>Priority: Major
> Fix For: 9.0.0
>
>
> Add R binding for {{not_between()}} compute function from ARROW-15223.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)


[jira] [Commented] (ARROW-15224) [R] Add binding for not_between() ternary kernel

2022-04-22 Thread Eduardo Ponce (Jira)


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

Eduardo Ponce commented on ARROW-15224:
---

For C++ ([ARROW-15223|https://issues.apache.org/jira/browse/ARROW-15223]) the 
{{not_between}} function was decided not to be included because it is 
equivalent to applying a logical {{NOT}} to the result of the {{BETWEEN}} 
function which is a simple composition to do. Also, {{not_between}} is not a 
common function in other DB/dataframe APIs.

> [R] Add binding for not_between() ternary kernel
> 
>
> Key: ARROW-15224
> URL: https://issues.apache.org/jira/browse/ARROW-15224
> Project: Apache Arrow
>  Issue Type: New Feature
>  Components: R
>Reporter: Eduardo Ponce
>Assignee: Dragoș Moldovan-Grünfeld
>Priority: Major
> Fix For: 9.0.0
>
>
> Add R binding for {{not_between()}} compute function from ARROW-15223.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)


[jira] [Commented] (ARROW-15224) [R] Add binding for not_between() ternary kernel

2022-04-22 Thread Jira


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

Dragoș Moldovan-Grünfeld commented on ARROW-15224:
--

Given {{dplyr::not_between()}} does not exist, do we need an R {{not_between()} 
binding? What do you think? [~jonkeane] [~thisisnic][~paleolimbot]

> [R] Add binding for not_between() ternary kernel
> 
>
> Key: ARROW-15224
> URL: https://issues.apache.org/jira/browse/ARROW-15224
> Project: Apache Arrow
>  Issue Type: New Feature
>  Components: R
>Reporter: Eduardo Ponce
>Assignee: Dragoș Moldovan-Grünfeld
>Priority: Major
> Fix For: 9.0.0
>
>
> Add R binding for {{not_between()}} compute function from ARROW-15223.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)