[
https://issues.apache.org/jira/browse/ARROW-11990?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17311162#comment-17311162
]
Yibo Cai commented on ARROW-11990:
----------------------------------
Hi [~bkietz], are you working on this jira?
I would like to do this task if you're not already on it.
> [C++][Compute] Use Status/Result return consistently to indicate errors
> -----------------------------------------------------------------------
>
> Key: ARROW-11990
> URL: https://issues.apache.org/jira/browse/ARROW-11990
> Project: Apache Arrow
> Issue Type: Improvement
> Components: C++
> Affects Versions: 1.0.0
> Reporter: Ben Kietzman
> Priority: Major
>
> Discussion on ML:
> https://lists.apache.org/thread.html/r7c369390dd0b99edb1e4529920e06e62a1f65802bcdfb34317436fac%40%3Cdev.arrow.apache.org%3E
> Wherever there is not a motivating performance reason for error
> propagation using {{KernelContext::{SetStatus, status}}} remove this in favor
> of the error pattern used in the rest of the library: returning a Status or
> (preferably) Result. This will avoid the added cognitive overhead and
> boilerplate code of interfacing between the different strategies.
> For example, refactor KernelInit from
> {{unique_ptr<KernelState>(KernelContext*, KernelInitArgs)}} to
> {{Result<unique_ptr<KernelState>>(ExecContext*, KernelInitiArgs)}}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
