[FLINK-17501] Improve logging in AbstractServerHandler#channelRead(ChannelHandlerContext, Object)

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.11.0
• Component/s: Runtime / Queryable State
• Labels: pull-request-available
• Assignee: Gary Yao
• Reporter: Gary Yao
• Created: 04/May/20 10:28
• Updated: 06/May/20 08:13
• Resolved: 06/May/20 08:13

Description

Improve logging in AbstractServerHandler#channelRead(ChannelHandlerContext, Object). If an Error is thrown, it should be logged as early as possible. Currently we try to serialize and send an error response to the client before logging the error; this can fail and mask the original exception.

Issue Links

• is related to: FLINK-13553 KvStateServerHandlerTest.readInboundBlocking unstable on Travis
• links to: GitHub Pull Request #11984