[jira] [Commented] (FLINK-7710) Port CheckpointStatsHandler to new REST endpoint

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

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

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

GitHub user tillrohrmann opened a pull request:

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

    [FLINK-7710] [flip6] Add CheckpointStatisticsHandler for the new REST endpoint

    ## What is the purpose of the change
    
    This commit also makes the CheckpointStatsHistory object serializable by removing the
    CheckpointStatsHistoryIterable and replacing it with a static ArrayList.
    
    ## Verifying this change
    
    *(Please pick either of the following options)*
    
    This change added tests and can be verified as follows:
    
    - `CheckpointStatisticsTest` which tests the marshalling of `CheckpointStatistics` instances
    
    ## 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 portCheckpointStatsHandler

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

    https://github.com/apache/flink/pull/4750.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 #4750
    
----
commit d6be27aac5d7ff022dc03874a8bb52b1c09d49de
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 29f5d7e2484d52889230c559031cb7c427173927
Author: Till Rohrmann <tr...@apache.org>
Date:   2017-09-26T16:39:15Z

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

commit e77fb7a5d78be540116051b9914cd70e61cf5df7
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.

commit cd32570761c129da0bfff1fe0a4b4af6ae193c9c
Author: Till Rohrmann <tr...@apache.org>
Date:   2017-09-29T13:09:06Z

    [FLINK-7710] [flip6] Add CheckpointStatisticsHandler for the new REST endpoint
    
    This commit also makes the CheckpointStatsHistory object serializable by removing the
    CheckpointStatsHistoryIterable and replacing it with a static ArrayList.

----


> Port CheckpointStatsHandler to new REST endpoint
> ------------------------------------------------
>
>                 Key: FLINK-7710
>                 URL: https://issues.apache.org/jira/browse/FLINK-7710
>             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 {{CheckpointStatsHandler}} to new REST endpoint.



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