[FLINK-7869] Don't use PipelineErrorHandler on the client

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Done
• Component/s: Deployment / Mesos, Runtime / Coordination
• Assignee: Unassigned
• Reporter: Eron Wright
• Created: 19/Oct/17 04:29
• Updated: 09/Oct/20 12:35
• Resolved: 09/Oct/20 12:35

Description

The `PipelineErrorHandler` is being used in both the server and client pipeline, but produces an HTTP response upon error (which obviously doesn't make sense on the client).

I believe that the `AbstractRestHandler` is best suited to producing an HTTP error response. Keep in mind that the channel might not be in a state where an HTTP response makes sense, e.g. after a web socket has been handshaked. I suggest that the last handler only log and close the channel.

Issue Links

• is fixed by: FLINK-7521 Remove the 10MB limit from the current REST implementation.