[jira] [Updated] (ARROW-13370) [R] More special handling for known errors in arrow_eval
[
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
* Surface error messages that do not contain the "not supported in Arrow"
keywords
* Update unit tests
* Document updates in error-handling
*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*:
* 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).
> [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
> * Surface error messages that do not contain the "not supported in Arrow"
> keywords
> * Update unit tests
> * Document updates in error-handling
> *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)
[jira] [Updated] (ARROW-13370) [R] More special handling for known errors in arrow_eval
[
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)
[jira] [Updated] (ARROW-13370) [R] More special handling for known errors in arrow_eval
[ 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*: *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: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*: > *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)
[jira] [Updated] (ARROW-13370) [R] More special handling for known errors in arrow_eval
[ https://issues.apache.org/jira/browse/ARROW-13370?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dragoș Moldovan-Grünfeld updated ARROW-13370: - Fix Version/s: (was: 9.0.0) > [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 > > 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)
[jira] [Updated] (ARROW-13370) [R] More special handling for known errors in arrow_eval
[ https://issues.apache.org/jira/browse/ARROW-13370?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jonathan Keane updated ARROW-13370: --- 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: 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) raised 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 >Priority: Major > Fix For: 9.0.0 > > > 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.7#820007)
[jira] [Updated] (ARROW-13370) [R] More special handling for known errors in arrow_eval
[ https://issues.apache.org/jira/browse/ARROW-13370?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Raúl Cumplido updated ARROW-13370: -- Fix Version/s: 9.0.0 (was: 8.0.0) > [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 >Priority: Major > Fix For: 9.0.0 > > > 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) raised a classed exception (if > that's supported in all versions of R we support). -- This message was sent by Atlassian Jira (v8.20.7#820007)
[jira] [Updated] (ARROW-13370) [R] More special handling for known errors in arrow_eval
[ https://issues.apache.org/jira/browse/ARROW-13370?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alessandro Molina updated ARROW-13370: -- Fix Version/s: 8.0.0 (was: 7.0.0) > [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 >Priority: Major > Fix For: 8.0.0 > > > 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) raised a classed exception (if > that's supported in all versions of R we support). -- This message was sent by Atlassian Jira (v8.20.1#820001)
