[ 
https://issues.apache.org/jira/browse/FLINK-5559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15832015#comment-15832015
 ] 

ASF GitHub Bot commented on FLINK-5559:
---------------------------------------

Github user uce commented on the issue:

    https://github.com/apache/flink/pull/3172
  
    I agree that it is more appropriate to throw the `IOException`.


> queryable state: KvStateRequestSerializer#deserializeKeyAndNamespace() throws 
> an IOException without own failure message if deserialisation fails
> -------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: FLINK-5559
>                 URL: https://issues.apache.org/jira/browse/FLINK-5559
>             Project: Flink
>          Issue Type: Improvement
>          Components: Queryable State
>    Affects Versions: 1.2.0
>            Reporter: Nico Kruber
>            Assignee: Nico Kruber
>            Priority: Minor
>
> KvStateRequestSerializer#deserializeKeyAndNamespace() throws an IOException, 
> e.g. EOFException, if the deserialisation fails, e.g. there are not enough 
> available bytes.
> In these cases, it should instead also throw an IllegalArgumentException with 
> a message containing "This indicates a mismatch in the key/namespace 
> serializers used by the KvState instance and this access." as the other error 
> cases.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to