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

Neal Richardson commented on ARROW-17495:
-----------------------------------------

Per https://github.com/apache/arrow/blob/master/r/R/arrow-package.R#L64-L67, 
nse_funcs can't be fully populated until load time. 

Feel free to try removing it and seeing what breaks, and be sure to check 
benchmarks--I don't know how much overhead there is in re-creating a bunch of 
function definitions every time you arrow_eval, maybe it's a lot and maybe it 
isn't.

> [R] arrow_eval: do we need both nse_funcs and .cache$functions?
> ---------------------------------------------------------------
>
>                 Key: ARROW-17495
>                 URL: https://issues.apache.org/jira/browse/ARROW-17495
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: R
>    Affects Versions: 9.0.0
>            Reporter: Dragoș Moldovan-Grünfeld
>            Priority: Minor
>
> Currently we have 2 copies of the same information, once in {{nse_funcs}} and 
> once in {{{}.cache$functions{}}}. I wasn't able to figure out the reason for 
> this. Maybe I am missing something or maybe this is just legacy code that we 
> can update.   



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to