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

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

Github user tillrohrmann commented on a diff in the pull request:

    https://github.com/apache/flink/pull/945#discussion_r37978039
  
    --- Diff: 
flink-runtime/src/test/java/org/apache/flink/runtime/jobmanager/JobSubmitTest.java
 ---
    @@ -170,4 +170,43 @@ public void initializeOnMaster(ClassLoader loader) 
throws Exception {
                        fail(e.getMessage());
                }
        }
    +
    +   /**
    +    * Verifies failure when the client and job manager versions mismatch
    +    */
    +   @Test
    +   public void testFailureClientJobManagerVersionMismatch() {
    +           try {
    +                   // create a simple job graph
    +
    +                   JobVertex jobVertex = new JobVertex("Vertex that fails 
in initializeOnMaster") {
    +
    +                           @Override
    +                           public void initializeOnMaster(ClassLoader 
loader) throws Exception {
    +                                   throw new RuntimeException("test 
exception");
    --- End diff --
    
    Why using a faulty `JobVertex` here. This might also be the reason for the 
"expected" `JobExecutionException` then. And you don't check properly whether 
that's not the cause for it.


> Fail when actor versions don't match
> ------------------------------------
>
>                 Key: FLINK-2399
>                 URL: https://issues.apache.org/jira/browse/FLINK-2399
>             Project: Flink
>          Issue Type: Improvement
>          Components: JobManager, TaskManager
>    Affects Versions: 0.9, master
>            Reporter: Ufuk Celebi
>            Assignee: Sachin Goel
>            Priority: Minor
>             Fix For: 0.10
>
>
> Problem: there can be subtle errors when actors from different Flink versions 
> communicate with each other, for example when an old client (e.g. Flink 0.9) 
> communicates with a new JobManager (e.g. Flink 0.10-SNAPSHOT).
> We can check that the versions match on first communication between the 
> actors and fail if they don't match.



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

Reply via email to