[FLINK-5576] extend deserialization functions of KvStateRequestSerializer to detect unconsumed bytes

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.2.0
• Fix Version/s: 1.2.0, 1.3.0
• Component/s: Runtime / Queryable State
• Assignee: Nico Kruber
• Reporter: Nico Kruber
• Created: 19/Jan/17 14:40
• Updated: 22/Jan/17 14:57
• Resolved: 22/Jan/17 14:57

Description

KvStateRequestSerializer#deserializeValue and KvStateRequestSerializer#deserializeList both deserialize a given byte array. This is used by clients and unit tests and it is fair to assume that these byte arrays represent a complete value since we do not offer a method to continue reading from the middle of the array anyway. Therefore, we can treat unconsumed bytes as errors, e.g. from a wrong serializer being used, and throw a IOException with an appropriate failure message.

Issue Links

• links to: GitHub Pull Request #3174