Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.9.0, 3.0.0-alpha2
    • Component/s: scheduler
    • Labels:
      None
    • Target Version/s:
    • Hadoop Flags:
      Reviewed
    1. YARN-5649.1.patch
      31 kB
      Rohith Sharma K S
    2. YARN-5649.2.patch
      28 kB
      Rohith Sharma K S
    3. YARN-5649.3.patch
      34 kB
      Rohith Sharma K S
    4. YARN-5649.branch-2.0001.patch
      35 kB
      Rohith Sharma K S
    5. YARN-5649.branch-2.0002.patch
      34 kB
      Rohith Sharma K S
    6. YARN-5649.v0.patch
      7 kB
      Rohith Sharma K S

      Issue Links

        Activity

        Hide
        sunilg Sunil G added a comment -

        Thanks Rohith Sharma K S for raising the same.

        We also need read-only getter api's here. This can help in having a better UI w.r.t timeout

        For eg, if we can have some thing like below

        ApplicationTimeout 
             ==>  configuredTimeout
             ==>  timeToExpire
        

        This can help us getting a picture how much more time is pending for app to become expire if it is not completed. Thoughts?

        Show
        sunilg Sunil G added a comment - Thanks Rohith Sharma K S for raising the same. We also need read-only getter api's here. This can help in having a better UI w.r.t timeout For eg, if we can have some thing like below ApplicationTimeout ==> configuredTimeout ==> timeToExpire This can help us getting a picture how much more time is pending for app to become expire if it is not completed. Thoughts?
        Hide
        rohithsharma Rohith Sharma K S added a comment -

        Updating v0 patch without tests for initial approach review. And also we can decide REST endpoints and other discussions.

        Show
        rohithsharma Rohith Sharma K S added a comment - Updating v0 patch without tests for initial approach review. And also we can decide REST endpoints and other discussions.
        Hide
        rohithsharma Rohith Sharma K S added a comment -

        The below are the REST API's will be exposing for timeout. Thoughts?

        @GET and @PUT
        http://<rm http address:port>/ws/v1/cluster/apps/{appid}/timeout

        Show
        rohithsharma Rohith Sharma K S added a comment - The below are the REST API's will be exposing for timeout. Thoughts? @GET and @PUT http://<rm http address:port>/ws/v1/cluster/apps/{appid}/timeout
        Hide
        rohithsharma Rohith Sharma K S added a comment -

        Updated patch with following REST endpoints.
        GET - /apps/{appid}/timeout/{type}, Currently we are supporting only one timeout type i.e LIFETIME, so need not to expose list of timeouts as of now in REST. May be this we do if we support more timeouts down the line.
        PUT - /apps/{appid}\/timeout

        Show
        rohithsharma Rohith Sharma K S added a comment - Updated patch with following REST endpoints. GET - /apps/{appid}/timeout/{type} , Currently we are supporting only one timeout type i.e LIFETIME, so need not to expose list of timeouts as of now in REST. May be this we do if we support more timeouts down the line. PUT - /apps/{appid}\/timeout
        Hide
        hadoopqa Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 19s Docker mode activated.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 test4tests 0m 0s The patch appears to include 2 new or modified test files.
        +1 mvninstall 7m 57s trunk passed
        +1 compile 0m 43s trunk passed
        +1 checkstyle 0m 29s trunk passed
        +1 mvnsite 0m 50s trunk passed
        +1 mvneclipse 0m 21s trunk passed
        +1 findbugs 1m 16s trunk passed
        +1 javadoc 0m 29s trunk passed
        +1 mvninstall 0m 41s the patch passed
        +1 compile 0m 38s the patch passed
        +1 javac 0m 38s the patch passed
        -0 checkstyle 0m 24s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 7 new + 162 unchanged - 1 fixed = 169 total (was 163)
        +1 mvnsite 0m 46s the patch passed
        +1 mvneclipse 0m 18s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        +1 findbugs 1m 20s the patch passed
        +1 javadoc 0m 22s the patch passed
        -1 unit 50m 36s hadoop-yarn-server-resourcemanager in the patch failed.
        +1 asflicense 0m 20s The patch does not generate ASF License warnings.
        69m 22s



        Reason Tests
        Failed junit tests hadoop.yarn.server.resourcemanager.TestRMRestart
          hadoop.yarn.server.resourcemanager.TestRMHA
          hadoop.yarn.server.resourcemanager.webapp.TestRMWebAppFairScheduler



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:a9ad5d6
        JIRA Issue YARN-5649
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12839713/YARN-5649.1.patch
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
        uname Linux c45e2f64e56b 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:00:09 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / 7584fbf
        Default Java 1.8.0_111
        findbugs v3.0.0
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/13986/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        unit https://builds.apache.org/job/PreCommit-YARN-Build/13986/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13986/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/13986/console
        Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 19s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 2 new or modified test files. +1 mvninstall 7m 57s trunk passed +1 compile 0m 43s trunk passed +1 checkstyle 0m 29s trunk passed +1 mvnsite 0m 50s trunk passed +1 mvneclipse 0m 21s trunk passed +1 findbugs 1m 16s trunk passed +1 javadoc 0m 29s trunk passed +1 mvninstall 0m 41s the patch passed +1 compile 0m 38s the patch passed +1 javac 0m 38s the patch passed -0 checkstyle 0m 24s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 7 new + 162 unchanged - 1 fixed = 169 total (was 163) +1 mvnsite 0m 46s the patch passed +1 mvneclipse 0m 18s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 1m 20s the patch passed +1 javadoc 0m 22s the patch passed -1 unit 50m 36s hadoop-yarn-server-resourcemanager in the patch failed. +1 asflicense 0m 20s The patch does not generate ASF License warnings. 69m 22s Reason Tests Failed junit tests hadoop.yarn.server.resourcemanager.TestRMRestart   hadoop.yarn.server.resourcemanager.TestRMHA   hadoop.yarn.server.resourcemanager.webapp.TestRMWebAppFairScheduler Subsystem Report/Notes Docker Image:yetus/hadoop:a9ad5d6 JIRA Issue YARN-5649 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12839713/YARN-5649.1.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux c45e2f64e56b 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:00:09 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 7584fbf Default Java 1.8.0_111 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/13986/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/13986/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13986/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager Console output https://builds.apache.org/job/PreCommit-YARN-Build/13986/console Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
        Hide
        rohithsharma Rohith Sharma K S added a comment -

        Updated patch fixing test failure and some minor typo corrections.

        Show
        rohithsharma Rohith Sharma K S added a comment - Updated patch fixing test failure and some minor typo corrections.
        Hide
        hadoopqa Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 18s Docker mode activated.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 test4tests 0m 0s The patch appears to include 3 new or modified test files.
        +1 mvninstall 7m 22s trunk passed
        +1 compile 0m 33s trunk passed
        +1 checkstyle 0m 23s trunk passed
        +1 mvnsite 0m 40s trunk passed
        +1 mvneclipse 0m 18s trunk passed
        +1 findbugs 1m 4s trunk passed
        +1 javadoc 0m 23s trunk passed
        +1 mvninstall 0m 35s the patch passed
        +1 compile 0m 34s the patch passed
        +1 javac 0m 34s the patch passed
        -0 checkstyle 0m 21s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 3 new + 184 unchanged - 1 fixed = 187 total (was 185)
        +1 mvnsite 0m 41s the patch passed
        +1 mvneclipse 0m 16s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        +1 findbugs 1m 18s the patch passed
        +1 javadoc 0m 25s the patch passed
        -1 unit 48m 5s hadoop-yarn-server-resourcemanager in the patch failed.
        +1 asflicense 0m 18s The patch does not generate ASF License warnings.
        64m 55s



        Reason Tests
        Failed junit tests hadoop.yarn.server.resourcemanager.TestRMRestart



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:a9ad5d6
        JIRA Issue YARN-5649
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12840026/YARN-5649.2.patch
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
        uname Linux 04548f73013d 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:00:09 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / a926f89
        Default Java 1.8.0_111
        findbugs v3.0.0
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14020/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        unit https://builds.apache.org/job/PreCommit-YARN-Build/14020/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14020/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/14020/console
        Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 18s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 3 new or modified test files. +1 mvninstall 7m 22s trunk passed +1 compile 0m 33s trunk passed +1 checkstyle 0m 23s trunk passed +1 mvnsite 0m 40s trunk passed +1 mvneclipse 0m 18s trunk passed +1 findbugs 1m 4s trunk passed +1 javadoc 0m 23s trunk passed +1 mvninstall 0m 35s the patch passed +1 compile 0m 34s the patch passed +1 javac 0m 34s the patch passed -0 checkstyle 0m 21s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 3 new + 184 unchanged - 1 fixed = 187 total (was 185) +1 mvnsite 0m 41s the patch passed +1 mvneclipse 0m 16s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 1m 18s the patch passed +1 javadoc 0m 25s the patch passed -1 unit 48m 5s hadoop-yarn-server-resourcemanager in the patch failed. +1 asflicense 0m 18s The patch does not generate ASF License warnings. 64m 55s Reason Tests Failed junit tests hadoop.yarn.server.resourcemanager.TestRMRestart Subsystem Report/Notes Docker Image:yetus/hadoop:a9ad5d6 JIRA Issue YARN-5649 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12840026/YARN-5649.2.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 04548f73013d 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:00:09 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / a926f89 Default Java 1.8.0_111 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14020/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/14020/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14020/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager Console output https://builds.apache.org/job/PreCommit-YARN-Build/14020/console Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
        Hide
        jianhe Jian He added a comment -

        Rohith Sharma K S, the failed ut seems related, can you check?

        Show
        jianhe Jian He added a comment - Rohith Sharma K S , the failed ut seems related, can you check?
        Hide
        bibinchundatt Bibin A Chundatt added a comment -

        YARN-5548 is available to track the same

        Show
        bibinchundatt Bibin A Chundatt added a comment - YARN-5548 is available to track the same
        Hide
        jianhe Jian He added a comment -

        I see, thanks.
        then this patch is good to go

        Show
        jianhe Jian He added a comment - I see, thanks. then this patch is good to go
        Hide
        bibinchundatt Bibin A Chundatt added a comment -

        Rohith Sharma K S/Jian He
        Over all patch looks good to me too.
        Instead of throwing YarnException during PUT better to return BadRequest rt??
        IIUC YarnException the return code would be 500 Internal Server Error when appTimeout is empty or null .

        Show
        bibinchundatt Bibin A Chundatt added a comment - Rohith Sharma K S / Jian He Over all patch looks good to me too. Instead of throwing YarnException during PUT better to return BadRequest rt?? IIUC YarnException the return code would be 500 Internal Server Error when appTimeout is empty or null .
        Hide
        rohithsharma Rohith Sharma K S added a comment -

        Instead of throwing YarnException during PUT better to return BadRequest rt??

        Few scenarios we can return BadRequest like if timeout type is empty. Few cases, it is required to throw YarnException I believe. Let me cross check with web services experts.

        Show
        rohithsharma Rohith Sharma K S added a comment - Instead of throwing YarnException during PUT better to return BadRequest rt?? Few scenarios we can return BadRequest like if timeout type is empty. Few cases, it is required to throw YarnException I believe. Let me cross check with web services experts.
        Hide
        vinodkv Vinod Kumar Vavilapalli added a comment -

        How about the following?

        • POST is meaningless as implicitly there is always a timeouts object for an app
        • GET
          • /apps/$appid/timeouts/ - Returns the collection of timeouts
          • /apps/$appid/timeouts/$type - Returns a timeout of a specific type
        • PUT
          • /apps/$appid/timeouts/$type - Update only one timeout value
          • /apps/$appid/timeouts/ - Replace the entire object API - too many scenarios here, with partial updates etc. Don't accept this request? Or is "not implementing it == not accepting the request"?
        Show
        vinodkv Vinod Kumar Vavilapalli added a comment - How about the following? POST is meaningless as implicitly there is always a timeouts object for an app GET /apps/$appid/timeouts/ - Returns the collection of timeouts /apps/$appid/timeouts/$type - Returns a timeout of a specific type PUT /apps/$appid/timeouts/$type - Update only one timeout value /apps/$appid/timeouts/ - Replace the entire object API - too many scenarios here, with partial updates etc. Don't accept this request? Or is "not implementing it == not accepting the request"?
        Hide
        rohithsharma Rohith Sharma K S added a comment -

        Updating the patch adding new GET api for collections. Rest of all remains same. For PUT, /timeouts is not implemented now.

        Show
        rohithsharma Rohith Sharma K S added a comment - Updating the patch adding new GET api for collections. Rest of all remains same. For PUT, /timeouts is not implemented now.
        Hide
        hadoopqa Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 20s Docker mode activated.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 test4tests 0m 0s The patch appears to include 3 new or modified test files.
        +1 mvninstall 9m 11s trunk passed
        +1 compile 0m 47s trunk passed
        +1 checkstyle 0m 26s trunk passed
        +1 mvnsite 0m 49s trunk passed
        +1 mvneclipse 0m 21s trunk passed
        +1 findbugs 1m 9s trunk passed
        +1 javadoc 0m 27s trunk passed
        +1 mvninstall 0m 34s the patch passed
        +1 compile 0m 33s the patch passed
        +1 javac 0m 33s the patch passed
        -0 checkstyle 0m 22s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 3 new + 227 unchanged - 1 fixed = 230 total (was 228)
        +1 mvnsite 0m 38s the patch passed
        +1 mvneclipse 0m 15s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        +1 findbugs 1m 15s the patch passed
        +1 javadoc 0m 19s the patch passed
        -1 unit 42m 21s hadoop-yarn-server-resourcemanager in the patch failed.
        +1 asflicense 0m 16s The patch does not generate ASF License warnings.
        61m 27s



        Reason Tests
        Failed junit tests hadoop.yarn.server.resourcemanager.TestRMRestart
          hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:a9ad5d6
        JIRA Issue YARN-5649
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12840298/YARN-5649.3.patch
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
        uname Linux 2352cf43d527 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:00:09 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / 005850b
        Default Java 1.8.0_111
        findbugs v3.0.0
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14065/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        unit https://builds.apache.org/job/PreCommit-YARN-Build/14065/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14065/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/14065/console
        Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 20s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 3 new or modified test files. +1 mvninstall 9m 11s trunk passed +1 compile 0m 47s trunk passed +1 checkstyle 0m 26s trunk passed +1 mvnsite 0m 49s trunk passed +1 mvneclipse 0m 21s trunk passed +1 findbugs 1m 9s trunk passed +1 javadoc 0m 27s trunk passed +1 mvninstall 0m 34s the patch passed +1 compile 0m 33s the patch passed +1 javac 0m 33s the patch passed -0 checkstyle 0m 22s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 3 new + 227 unchanged - 1 fixed = 230 total (was 228) +1 mvnsite 0m 38s the patch passed +1 mvneclipse 0m 15s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 1m 15s the patch passed +1 javadoc 0m 19s the patch passed -1 unit 42m 21s hadoop-yarn-server-resourcemanager in the patch failed. +1 asflicense 0m 16s The patch does not generate ASF License warnings. 61m 27s Reason Tests Failed junit tests hadoop.yarn.server.resourcemanager.TestRMRestart   hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer Subsystem Report/Notes Docker Image:yetus/hadoop:a9ad5d6 JIRA Issue YARN-5649 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12840298/YARN-5649.3.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 2352cf43d527 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:00:09 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 005850b Default Java 1.8.0_111 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14065/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/14065/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14065/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager Console output https://builds.apache.org/job/PreCommit-YARN-Build/14065/console Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
        Hide
        jianhe Jian He added a comment -

        looks good, committing

        Show
        jianhe Jian He added a comment - looks good, committing
        Hide
        jianhe Jian He added a comment -

        Committed to trunk, branch-2, thanks Rohith !

        Thanks Sunil, Vinod for reviewing !

        Show
        jianhe Jian He added a comment - Committed to trunk, branch-2, thanks Rohith ! Thanks Sunil, Vinod for reviewing !
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10882 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10882/)
        YARN-5649. Add REST endpoints for updating application timeouts. (jianhe: rev 1f12867a69544a1642aa986d4f9a8249be495434)

        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/RMServerUtils.java
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppInfo.java
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ClientRMService.java
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/rmapp/MockRMApp.java
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesApps.java
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/RMAuditLogger.java
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java
        • (add) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppTimeoutsInfo.java
        • (add) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppTimeoutInfo.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10882 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10882/ ) YARN-5649 . Add REST endpoints for updating application timeouts. (jianhe: rev 1f12867a69544a1642aa986d4f9a8249be495434) (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/RMServerUtils.java (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppInfo.java (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ClientRMService.java (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/rmapp/MockRMApp.java (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesApps.java (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/RMAuditLogger.java (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java (add) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppTimeoutsInfo.java (add) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppTimeoutInfo.java
        Hide
        liuml07 Mingliang Liu added a comment -

        branch-2 is broken because of this?

        Show
        liuml07 Mingliang Liu added a comment - branch-2 is broken because of this?
        Hide
        rohithsharma Rohith Sharma K S added a comment -

        yes, let us revert the changes as many conflicts are there to resolve. I will upload a new patch for branch-2 again.

        Show
        rohithsharma Rohith Sharma K S added a comment - yes, let us revert the changes as many conflicts are there to resolve. I will upload a new patch for branch-2 again.
        Hide
        rohithsharma Rohith Sharma K S added a comment -

        Reopening the issue for handling revert, and uploading a new patch to branch-2.

        Show
        rohithsharma Rohith Sharma K S added a comment - Reopening the issue for handling revert, and uploading a new patch to branch-2.
        Hide
        rohithsharma Rohith Sharma K S added a comment -

        Updated branch-2 patch. cc:/Jian He

        Show
        rohithsharma Rohith Sharma K S added a comment - Updated branch-2 patch. cc:/ Jian He
        Hide
        jianhe Jian He added a comment -

        the branch-2 patch looks good to me.

        Show
        jianhe Jian He added a comment - the branch-2 patch looks good to me.
        Hide
        hadoopqa Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 16s Docker mode activated.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 test4tests 0m 0s The patch appears to include 3 new or modified test files.
        +1 mvninstall 6m 26s branch-2 passed
        +1 compile 0m 29s branch-2 passed with JDK v1.8.0_111
        +1 compile 0m 32s branch-2 passed with JDK v1.7.0_111
        +1 checkstyle 0m 26s branch-2 passed
        +1 mvnsite 0m 38s branch-2 passed
        +1 mvneclipse 0m 17s branch-2 passed
        +1 findbugs 1m 11s branch-2 passed
        +1 javadoc 0m 20s branch-2 passed with JDK v1.8.0_111
        +1 javadoc 0m 25s branch-2 passed with JDK v1.7.0_111
        +1 mvninstall 0m 32s the patch passed
        +1 compile 0m 25s the patch passed with JDK v1.8.0_111
        +1 javac 0m 25s the patch passed
        +1 compile 0m 30s the patch passed with JDK v1.7.0_111
        +1 javac 0m 30s the patch passed
        -0 checkstyle 0m 23s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 3 new + 233 unchanged - 0 fixed = 236 total (was 233)
        +1 mvnsite 0m 35s the patch passed
        +1 mvneclipse 0m 14s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        +1 findbugs 1m 22s the patch passed
        +1 javadoc 0m 17s the patch passed with JDK v1.8.0_111
        +1 javadoc 0m 22s the patch passed with JDK v1.7.0_111
        -1 unit 39m 56s hadoop-yarn-server-resourcemanager in the patch failed with JDK v1.7.0_111.
        +1 asflicense 0m 18s The patch does not generate ASF License warnings.
        95m 8s



        Reason Tests
        JDK v1.8.0_111 Failed junit tests hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesAppsModification
          hadoop.yarn.server.resourcemanager.TestRMRestart
        JDK v1.7.0_111 Failed junit tests hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer
          hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesAppsModification



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:b59b8b7
        JIRA Issue YARN-5649
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12840372/YARN-5649.branch-2.0001.patch
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
        uname Linux f46eca5b18f1 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision branch-2 / 6b47a7f
        Default Java 1.7.0_111
        Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_111 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_111
        findbugs v3.0.0
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14073/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        unit https://builds.apache.org/job/PreCommit-YARN-Build/14073/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdk1.7.0_111.txt
        JDK v1.7.0_111 Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14073/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/14073/console
        Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 16s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 3 new or modified test files. +1 mvninstall 6m 26s branch-2 passed +1 compile 0m 29s branch-2 passed with JDK v1.8.0_111 +1 compile 0m 32s branch-2 passed with JDK v1.7.0_111 +1 checkstyle 0m 26s branch-2 passed +1 mvnsite 0m 38s branch-2 passed +1 mvneclipse 0m 17s branch-2 passed +1 findbugs 1m 11s branch-2 passed +1 javadoc 0m 20s branch-2 passed with JDK v1.8.0_111 +1 javadoc 0m 25s branch-2 passed with JDK v1.7.0_111 +1 mvninstall 0m 32s the patch passed +1 compile 0m 25s the patch passed with JDK v1.8.0_111 +1 javac 0m 25s the patch passed +1 compile 0m 30s the patch passed with JDK v1.7.0_111 +1 javac 0m 30s the patch passed -0 checkstyle 0m 23s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 3 new + 233 unchanged - 0 fixed = 236 total (was 233) +1 mvnsite 0m 35s the patch passed +1 mvneclipse 0m 14s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 1m 22s the patch passed +1 javadoc 0m 17s the patch passed with JDK v1.8.0_111 +1 javadoc 0m 22s the patch passed with JDK v1.7.0_111 -1 unit 39m 56s hadoop-yarn-server-resourcemanager in the patch failed with JDK v1.7.0_111. +1 asflicense 0m 18s The patch does not generate ASF License warnings. 95m 8s Reason Tests JDK v1.8.0_111 Failed junit tests hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesAppsModification   hadoop.yarn.server.resourcemanager.TestRMRestart JDK v1.7.0_111 Failed junit tests hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer   hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesAppsModification Subsystem Report/Notes Docker Image:yetus/hadoop:b59b8b7 JIRA Issue YARN-5649 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12840372/YARN-5649.branch-2.0001.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux f46eca5b18f1 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision branch-2 / 6b47a7f Default Java 1.7.0_111 Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_111 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_111 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14073/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/14073/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdk1.7.0_111.txt JDK v1.7.0_111 Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14073/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager Console output https://builds.apache.org/job/PreCommit-YARN-Build/14073/console Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
        Hide
        sunilg Sunil G added a comment -

        +1 to latest branch-2 patch. It compiles on my branch-2. Pending jenkins.

        Show
        sunilg Sunil G added a comment - +1 to latest branch-2 patch. It compiles on my branch-2. Pending jenkins.
        Hide
        hadoopqa Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 18m 52s Docker mode activated.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 test4tests 0m 0s The patch appears to include 3 new or modified test files.
        +1 mvninstall 8m 20s branch-2 passed
        +1 compile 0m 34s branch-2 passed with JDK v1.8.0_111
        +1 compile 0m 39s branch-2 passed with JDK v1.7.0_121
        +1 checkstyle 0m 32s branch-2 passed
        +1 mvnsite 0m 46s branch-2 passed
        +1 mvneclipse 0m 21s branch-2 passed
        +1 findbugs 1m 27s branch-2 passed
        +1 javadoc 0m 32s branch-2 passed with JDK v1.8.0_111
        +1 javadoc 0m 32s branch-2 passed with JDK v1.7.0_121
        +1 mvninstall 0m 42s the patch passed
        +1 compile 0m 32s the patch passed with JDK v1.8.0_111
        +1 javac 0m 32s the patch passed
        +1 compile 0m 35s the patch passed with JDK v1.7.0_121
        +1 javac 0m 35s the patch passed
        -0 checkstyle 0m 26s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 3 new + 232 unchanged - 1 fixed = 235 total (was 233)
        +1 mvnsite 0m 42s the patch passed
        +1 mvneclipse 0m 16s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        +1 findbugs 1m 32s the patch passed
        +1 javadoc 0m 24s the patch passed with JDK v1.8.0_111
        +1 javadoc 0m 26s the patch passed with JDK v1.7.0_121
        -1 unit 45m 3s hadoop-yarn-server-resourcemanager in the patch failed with JDK v1.7.0_121.
        +1 asflicense 0m 17s The patch does not generate ASF License warnings.
        132m 48s



        Reason Tests
        JDK v1.8.0_111 Failed junit tests hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer
        JDK v1.7.0_121 Failed junit tests hadoop.yarn.server.resourcemanager.TestRMRestart



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:b59b8b7
        JIRA Issue YARN-5649
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12840395/YARN-5649.branch-2.0002.patch
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
        uname Linux f82ad306ebc6 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:00:09 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision branch-2 / 6b47a7f
        Default Java 1.7.0_121
        Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_111 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_121
        findbugs v3.0.0
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14076/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        unit https://builds.apache.org/job/PreCommit-YARN-Build/14076/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdk1.7.0_121.txt
        JDK v1.7.0_121 Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14076/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/14076/console
        Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 reexec 18m 52s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 3 new or modified test files. +1 mvninstall 8m 20s branch-2 passed +1 compile 0m 34s branch-2 passed with JDK v1.8.0_111 +1 compile 0m 39s branch-2 passed with JDK v1.7.0_121 +1 checkstyle 0m 32s branch-2 passed +1 mvnsite 0m 46s branch-2 passed +1 mvneclipse 0m 21s branch-2 passed +1 findbugs 1m 27s branch-2 passed +1 javadoc 0m 32s branch-2 passed with JDK v1.8.0_111 +1 javadoc 0m 32s branch-2 passed with JDK v1.7.0_121 +1 mvninstall 0m 42s the patch passed +1 compile 0m 32s the patch passed with JDK v1.8.0_111 +1 javac 0m 32s the patch passed +1 compile 0m 35s the patch passed with JDK v1.7.0_121 +1 javac 0m 35s the patch passed -0 checkstyle 0m 26s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 3 new + 232 unchanged - 1 fixed = 235 total (was 233) +1 mvnsite 0m 42s the patch passed +1 mvneclipse 0m 16s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 1m 32s the patch passed +1 javadoc 0m 24s the patch passed with JDK v1.8.0_111 +1 javadoc 0m 26s the patch passed with JDK v1.7.0_121 -1 unit 45m 3s hadoop-yarn-server-resourcemanager in the patch failed with JDK v1.7.0_121. +1 asflicense 0m 17s The patch does not generate ASF License warnings. 132m 48s Reason Tests JDK v1.8.0_111 Failed junit tests hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer JDK v1.7.0_121 Failed junit tests hadoop.yarn.server.resourcemanager.TestRMRestart Subsystem Report/Notes Docker Image:yetus/hadoop:b59b8b7 JIRA Issue YARN-5649 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12840395/YARN-5649.branch-2.0002.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux f82ad306ebc6 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:00:09 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision branch-2 / 6b47a7f Default Java 1.7.0_121 Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_111 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_121 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14076/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/14076/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdk1.7.0_121.txt JDK v1.7.0_121 Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14076/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager Console output https://builds.apache.org/job/PreCommit-YARN-Build/14076/console Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
        Hide
        sunilg Sunil G added a comment -
        YARN-5816 TestDelegationTokenRenewer.testCancelWithMultipleAppSubmissions
        YARN-5548 TestRMRestart.testFinishedAppRemovalAfterRMRestart

        These test cases failures are already tracked by mentioned jiras. Committing the patch.

        Show
        sunilg Sunil G added a comment - YARN-5816 TestDelegationTokenRenewer.testCancelWithMultipleAppSubmissions YARN-5548 TestRMRestart.testFinishedAppRemovalAfterRMRestart These test cases failures are already tracked by mentioned jiras. Committing the patch.
        Hide
        sunilg Sunil G added a comment -

        Committed to branch-2. Thanks Rohith Sharma K S for providing the patch. Thanks Jian He for additional reviews and thanks Mingliang Liu for helping to report this.

        Show
        sunilg Sunil G added a comment - Committed to branch-2. Thanks Rohith Sharma K S for providing the patch. Thanks Jian He for additional reviews and thanks Mingliang Liu for helping to report this.
        Hide
        rohithsharma Rohith Sharma K S added a comment -

        Thanks Sunil for committing the patch.

        Show
        rohithsharma Rohith Sharma K S added a comment - Thanks Sunil for committing the patch.

          People

          • Assignee:
            rohithsharma Rohith Sharma K S
            Reporter:
            rohithsharma Rohith Sharma K S
          • Votes:
            0 Vote for this issue
            Watchers:
            9 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development