[jira] [Commented] (ARROW-16068) [C++][FlightRPC] Migrate remaining flight API to use Result<>

["Tobias Zagorni (Jira)" <ji...@apache.org>]

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

Tobias Zagorni commented on ARROW-16068:
----------------------------------------

I mostly had two things in mind with this, which I'm not sure about changing them really makes sense looking what these methods do:
 * RecordBatchStream, started converting the Next() function there for consistency in ARROW-15612 but it is all virtual, so maybe better not convert more (I still need to fix a small problem I noticed with the next function)
 * PeekableFlightDataReader could probably be converted, but is a completely internal API so maybe out of scope here

> [C++][FlightRPC] Migrate remaining flight API to use Result<>
> -------------------------------------------------------------
>
>                 Key: ARROW-16068
>                 URL: https://issues.apache.org/jira/browse/ARROW-16068
>             Project: Apache Arrow
>          Issue Type: Sub-task
>          Components: C++, FlightRPC
>            Reporter: Tobias Zagorni
>            Assignee: Tobias Zagorni
>            Priority: Major
>




--
This message was sent by Atlassian Jira
(v8.20.1#820001)