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

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

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

    https://github.com/apache/flink/pull/1642#discussion_r53023536
  
    --- Diff: 
flink-runtime/src/test/java/org/apache/flink/runtime/checkpoint/SavepointCoordinatorTest.java
 ---
    @@ -337,19 +326,14 @@ public void testRollbackStateStoreFailure() throws 
Exception {
     
        @Test
        public void testRollbackUpdatesApplicationID() throws Exception {
    --- End diff --
    
    test method should renamed


> Remove ApplicationID from Environment
> -------------------------------------
>
>                 Key: FLINK-3299
>                 URL: https://issues.apache.org/jira/browse/FLINK-3299
>             Project: Flink
>          Issue Type: Improvement
>            Reporter: Ufuk Celebi
>            Assignee: Ufuk Celebi
>             Fix For: 1.0.0
>
>
> {{ApplicationID}} is used to identify an application across many job 
> submissions (for example after restoring from a savepoint). This is currently 
> exposed in the {{Environment}}, which might be unnecessary.
> State backends, which need the ID can generate it themselves and store it as 
> part of their state handle.
> This has to be checked with the DB state backend, which currently uses this.



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

Reply via email to