[ARROW-8879] [FlightRPC][Java] FlightStream should unwrap ExecutionExceptions

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Not A Problem
• Affects Version/s: 0.17.1
• Fix Version/s: 1.0.0
• Component/s: FlightRPC, Java
• External issue URL: https://github.com/apache/arrow/issues/25014
• Assignee: David Li
• Reporter: David Li
• Created: 21/May/20 12:19
• Updated: 11/Jan/23 08:03
• Resolved: 18/Aug/20 14:26

Description

Currently FlightStream bubbles a lot of exceptions as RuntimeException or ExecutionException, or just wraps them with CallStatus.INTERNAL. For RuntimeException, we should always check if it's a gRPC StatusRuntimeException and convert to the equivalent Flight exception; for ExecutionException, we should check if the cause is a gRPC exception and convert.

Example: on master, FlightStream#getDescriptor reports all errors as CallStatus.INTERNAL, but we should inspect ExecutionException#getCause instead.

This is needed so that errors get properly reported, e.g. if a service sends a PERMISSION_DENIED error, the client should get that and not a RuntimeException, ExecutionException, or INTERNAL error.