[GitHub] flink pull request #3972: [FLINK-6662] [errMsg] Improve error message if rec...

2017-05-23 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/flink/pull/3972


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] flink pull request #3972: [FLINK-6662] [errMsg] Improve error message if rec...

2017-05-23 Thread tillrohrmann
Github user tillrohrmann commented on a diff in the pull request:

https://github.com/apache/flink/pull/3972#discussion_r118068470
  
--- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/checkpoint/ZooKeeperCompletedCheckpointStore.java
 ---
@@ -376,8 +377,14 @@ private static CompletedCheckpoint 
retrieveCompletedCheckpoint(Tuple2

[GitHub] flink pull request #3972: [FLINK-6662] [errMsg] Improve error message if rec...

2017-05-23 Thread zentol
Github user zentol commented on a diff in the pull request:

https://github.com/apache/flink/pull/3972#discussion_r118026253
  
--- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/checkpoint/ZooKeeperCompletedCheckpointStore.java
 ---
@@ -376,8 +377,14 @@ private static CompletedCheckpoint 
retrieveCompletedCheckpoint(Tuple2

[GitHub] flink pull request #3972: [FLINK-6662] [errMsg] Improve error message if rec...

2017-05-23 Thread tillrohrmann
GitHub user tillrohrmann opened a pull request:

https://github.com/apache/flink/pull/3972

[FLINK-6662] [errMsg] Improve error message if recovery from 
RetrievableStateHandles fails

When recovering state from a ZooKeeperStateHandleStore it can happen that 
the deserialization
fails, because one tries to recover state from an old Flink version which 
is not compatible.
In this case we should output a better error message such that the user can 
easily spot the
problem.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/tillrohrmann/flink improveErrorMessages

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/flink/pull/3972.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #3972


commit 31d099c4768f1ee8dfbecfd8eddc6f05842425e6
Author: Till Rohrmann 
Date:   2017-05-23T13:42:38Z

[FLINK-6662] [errMsg] Improve error message if recovery from 
RetrievableStateHandles fails

When recovering state from a ZooKeeperStateHandleStore it can happen that 
the deserialization
fails, because one tries to recover state from an old Flink version which 
is not compatible.
In this case we should output a better error message such that the user can 
easily spot the
problem.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---