You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by uce <gi...@git.apache.org> on 2017/10/23 12:45:13 UTC

[GitHub] flink pull request #4888: [backport] [FLINK-7067] Resume checkpointing after...

GitHub user uce opened a pull request:

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

    [backport] [FLINK-7067] Resume checkpointing after failed cancel-job-with-savepoint

    This is a backport of #4254. I will merge this as soon as Travis gives the green light.


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

    $ git pull https://github.com/uce/flink 7067-backport

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

    https://github.com/apache/flink/pull/4888.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 #4888
    
----
commit 9226c3a15f8037851110fbdecf775cad99da771f
Author: Ufuk Celebi <uc...@apache.org>
Date:   2017-07-04T14:39:02Z

    [hotfix] [tests] Reduce visibility of helper class methods
    
    There is no need to make the helper methods public. No other class
    should even use this inner test helper invokable.

commit c571929ce476f17d02ee22df0b5170b0eb322c2d
Author: Ufuk Celebi <uc...@apache.org>
Date:   2017-07-04T15:01:32Z

    [FLINK-7067] [jobmanager] Resume periodic checkpoints after failed cancel-job-with-savepoint
    
    Problem: If a cancel-job-with-savepoint request fails, this has an
    unintended side effect on the respective job if it has periodic
    checkpoints enabled. The periodic checkpoint scheduler is stopped
    before triggering the savepoint, but not restarted if a savepoint
    fails and the job is not cancelled.
    
    This commit makes sure that the periodic checkpoint scheduler is
    restarted iff periodic checkpoints were enabled before.
    
    This closes #4254.

commit 074630a2fbd6dbdc7ff775ee9fb5d46c088dbc6d
Author: Ufuk Celebi <uc...@apache.org>
Date:   2017-10-23T12:42:46Z

    [FLINK-7067] [jobmanager] Backport to 1.3

----


---

[GitHub] flink issue #4888: [backport] [FLINK-7067] Resume checkpointing after failed...

Posted by uce <gi...@git.apache.org>.
Github user uce commented on the issue:

    https://github.com/apache/flink/pull/4888
  
    All failed tests are due to a similar issue that seems not to be related to the tests. Merging...


---

[GitHub] flink pull request #4888: [backport] [FLINK-7067] Resume checkpointing after...

Posted by uce <gi...@git.apache.org>.
Github user uce closed the pull request at:

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


---

[GitHub] flink issue #4888: [backport] [FLINK-7067] Resume checkpointing after failed...

Posted by uce <gi...@git.apache.org>.
Github user uce commented on the issue:

    https://github.com/apache/flink/pull/4888
  
    Merged in e8e2913.


---

[GitHub] flink issue #4888: [backport] [FLINK-7067] Resume checkpointing after failed...

Posted by uce <gi...@git.apache.org>.
Github user uce commented on the issue:

    https://github.com/apache/flink/pull/4888
  
    @aljoscha Getting the following message in two of the builds in https://travis-ci.org/apache/flink/builds/291523047:
    ```
    ==============================================================================
    Compilation/test failure detected, skipping shaded dependency check.
    ==============================================================================
    ```
    
    Ever seen this?


---