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

Nicola Crane commented on ARROW-17495:
--------------------------------------

[~dragosmg] I was trying to look into this, and saw this in the docs for the 
{{update_cache}} param for {{register_binding()}}:

"The reason why .cache$functions is needed in addition to nse_funcs for 
non-aggregate functions could be revisited...it is currently used as the data 
mask in mutate, filter, and aggregate (but not summarise) because the data mask 
has to be a list."

Given that [~paleolimbot] made the PR for the function registry stuff, he might 
be the person to ask about this?



> [R] arrow_eval: do we need both use_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: Major
>
> 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