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

Dragoș Moldovan-Grünfeld updated ARROW-13370:
---------------------------------------------
    Description: 
*Proposed approach*:
* Investigate the use of {{rlang::abort()}} with the class arg
* Investigate if an expansion of the {{arrow}} error classes is required
* 

*Original description*:
We have special handling in arrow_eval that looks for the "not supported in 
Arrow" error, and when that's found it shows the error message rather than 
swallowing it in an "Expression not supported" message. But we have other error 
messages we raise in nse_funcs that are worth showing--bad input etc. Use a 
sentinel error message that we can also detect and subclass as 
"arrow-try-error" like the others, or (better) raise a classed exception (if 
that's supported in all versions of R we support). 

  was:
*Proposed approach*:

*Original description*:
We have special handling in arrow_eval that looks for the "not supported in 
Arrow" error, and when that's found it shows the error message rather than 
swallowing it in an "Expression not supported" message. But we have other error 
messages we raise in nse_funcs that are worth showing--bad input etc. Use a 
sentinel error message that we can also detect and subclass as 
"arrow-try-error" like the others, or (better) raise a classed exception (if 
that's supported in all versions of R we support). 


> [R] More special handling for known errors in arrow_eval
> --------------------------------------------------------
>
>                 Key: ARROW-13370
>                 URL: https://issues.apache.org/jira/browse/ARROW-13370
>             Project: Apache Arrow
>          Issue Type: New Feature
>          Components: R
>            Reporter: Neal Richardson
>            Assignee: Dragoș Moldovan-Grünfeld
>            Priority: Major
>
> *Proposed approach*:
> * Investigate the use of {{rlang::abort()}} with the class arg
> * Investigate if an expansion of the {{arrow}} error classes is required
> * 
> *Original description*:
> We have special handling in arrow_eval that looks for the "not supported in 
> Arrow" error, and when that's found it shows the error message rather than 
> swallowing it in an "Expression not supported" message. But we have other 
> error messages we raise in nse_funcs that are worth showing--bad input etc. 
> Use a sentinel error message that we can also detect and subclass as 
> "arrow-try-error" like the others, or (better) raise a classed exception (if 
> that's supported in all versions of R we support). 



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

Reply via email to