[ARROW-15612] [C++] Migrate Flight APIs to Result<>

• Type: Task
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Component/s: C++, FlightRPC
• Labels: good-first-issue
• External issue URL: https://github.com/apache/arrow/issues/31074
• Assignee: Unassigned
• Reporter: Antoine Pitrou
• Created: 08/Feb/22 13:13
• Updated: 11/Jan/23 11:38

Description

A number of Flight APIs use the convention of returning a Status and filling an actual return value in a pointer-out (T*) parameter. They should be migrated to returning Result<T> instead, together with a deprecation notice on the legacy API.

Examples are BasicAuth::Serialize, BasicAuth::Deserialize, Location::Parse, etc.

Sub-Tasks

• [C++] Migrate flight/types.h header definitions to use Result<>: Resolved
• [C++] Migrate FlightClient API to Result<>: Resolved
• [C++] Migrate remaining flight API to use Resut<>: Closed
• [C++][FlightRPC] Migrate remaining flight API to use Result<>: Resolved
• [C++][FlightRPC] Migrate SQL Client API to Result<>: Open