[jira] [Commented] (FLINK-7708) Port CheckpointConfigHandler to new REST endpoint

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

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

GitHub user tillrohrmann opened a pull request:

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

    [FLINK-7708] [flip6] Add CheckpointConfigHandler for new REST endpoint

    ## What is the purpose of the change
    
    This commit implements the CheckpointConfigHandler which now returns a
    CheckpointConfigInfo object if checkpointing is enabled. In case that
    checkpointing is disabled for a job, it will return a 404 response.
    
    ## Verifying this change
    
    This change added tests and can be verified as follows:
    
    - `CheckpointConfigInfoTest` checks the marshalling of `CheckpointConfigInfo`
    
    ## Does this pull request potentially affect one of the following parts:
    
      - Dependencies (does it add or upgrade a dependency): (no)
      - The public API, i.e., is any changed class annotated with `@Public(Evolving)`: (no)
      - The serializers: (no)
      - The runtime per-record code paths (performance sensitive): (no)
      - Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
    
    ## Documentation
    
      - Does this pull request introduce a new feature? (no)
      - If yes, how is the feature documented? (not applicable)
    


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

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

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

    https://github.com/apache/flink/pull/4744.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 #4744
    
----
commit 6edb0151e9206d024f1b9437324d38988a762da5
Author: Till Rohrmann <tr...@apache.org>
Date:   2017-09-22T11:31:12Z

    [FLINK-7667] [flip6] Use ArchivedExecutionGraph as serializable AccessExecutionGraph
    
    This commit removes AccessExecutionGraph#getCheckpointCoordinator and changes the
    AccessExecutionGraph#getJobCheckpointSettings into #getJobCheckpointConfiguration.
    The JobCheckpointConfiguration only contains the CheckpointCoordinator relevant
    configuration settings and excludes the serialized state backend and the
    serialized master hooks. That way we don't send unnecessary information when
    the ArchivedExecutionGraph is requested.
    
    This closes #4727.

commit 33144cb4cd7c9a15fa8dde3a14ba2c66ffbf8559
Author: Till Rohrmann <tr...@apache.org>
Date:   2017-09-25T13:29:59Z

    [FLINK-7668] Add ExecutionGraphCache for ExecutionGraph based REST handlers
    
    The ExecutionGraphCache replaces the ExecutionGraphHolder. Unlike the latter, the former
    does not expect the AccessExecutionGraph to be the true ExecutionGraph. Instead it assumes
    it to be the ArchivedExecutionGraph. Therefore, it invalidates the cache entries after
    a given time to live period. This will trigger requesting the AccessExecutionGraph again
    and, thus, updating the ExecutionGraph information for the ExecutionGraph based REST
    handlers.
    
    In order to avoid memory leaks, the WebRuntimeMonitor starts now a periodic cleanup task
    which triggers ExecutionGraphCache.cleanup. This methods releases all cache entries which
    have exceeded their time to live. Currently it is set to 20 * refreshInterval of the
    web gui.
    
    This closes #4728.

commit da61847f313be2b18ad1f6f0bc0d827ba84acc5e
Author: Till Rohrmann <tr...@apache.org>
Date:   2017-09-26T16:39:15Z

    [FLINK-7695] [flip6] Add JobConfigHandler for new RestServerEndpoint

commit 76ff83338ef9ac0c794fb6cb4889abc8aea12577
Author: Till Rohrmann <tr...@apache.org>
Date:   2017-09-28T16:35:50Z

    [FLINK-7708] [flip6] Add CheckpointConfigHandler for new REST endpoint
    
    This commit implements the CheckpointConfigHandler which now returns a
    CheckpointConfigInfo object if checkpointing is enabled. In case that
    checkpointing is disabled for a job, it will return a 404 response.

----


> Port CheckpointConfigHandler to new REST endpoint
> -------------------------------------------------
>
>                 Key: FLINK-7708
>                 URL: https://issues.apache.org/jira/browse/FLINK-7708
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Distributed Coordination, REST, Webfrontend
>            Reporter: Tzu-Li (Gordon) Tai
>            Assignee: Till Rohrmann
>              Labels: flip-6
>             Fix For: 1.4.0
>
>
> Port existing {{CheckpointConfigHandler}} to new REST endpoint.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)