Uploaded image for project: 'Hadoop YARN'
  1. Hadoop YARN
  2. YARN-6093

Minor bugs with AMRMtoken renewal and state store availability when using FederationRMFailoverProxyProvider during RM failover

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: YARN-2915
    • Fix Version/s: 3.0.0-beta1
    • Component/s: amrmproxy, federation
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      AMRMProxy uses expired AMRMToken to talk to RM, leading to the "Invalid AMRMToken" exception. The bug is triggered when both conditions are met:
      1. RM rolls master key and renews AMRMToken for a running AM.
      2. Existing RPC connection between AMRMProxy and RM drops and attempt to reconnect via failover in FederationRMFailoverProxyProvider.

      Here's what happened:

      In DefaultRequestInterceptor.init(), we create a proxy ugi, load it with the initial AMRMToken issued by RM, and used it for initiating rmClient. Then we arrive at FederationRMFailoverProxyProvider.init(), a full copy of ugi tokens are saved locally, create an actual RM proxy and setup the RPC connection.

      Later when RM rolls master key and issues a new AMRMToken, DefaultRequestInterceptor.updateAMRMToken() updates it into the proxy ugi.

      However the new token is never used, until the existing RPC connection between AMRMProxy and RM drops for other reasons (say master RM crashes).

      When we try to reconnect, since the service name of the new AMRMToken is not yet set correctly in DefaultRequestInterceptor.updateAMRMToken(), RPC found no valid AMRMToken when trying to setup a new connection. We first hit a "Client cannot authenticate via:[TOKEN]" exception. This is expected.

      Next, FederationRMFailoverProxyProvider fails over, we reset the service token via ClientRMProxy.getRMAddress() and reconnect. Supposedly this would have worked.

      However since DefaultRequestInterceptor does not use the proxy user for later calls to rmClient, when performing failover in FederationRMFailoverProxyProvider, we are not in the proxy user. Currently the code solve the problem by reloading the current ugi with all tokens saved locally in originalTokens in method addOriginalTokens(). The problem is that the original AMRMToken loaded is no longer accepted by RM, and thus we keep hitting the "Invalid AMRMToken" exception until AM fails.

      The correct way is that rather than saving the original tokens in the proxy ugi, we save the original ugi itself. Every time we perform failover and create the new RM proxy, we use the original ugi, which is always loaded with the up-to-date AMRMToken.

      1. YARN-6093.v1.patch
        10 kB
        Botong Huang
      2. YARN-6093-08dc09581230ba595ce48fe7d3bc4eb2b6f98091.v4.patch
        10 kB
        Subru Krishnan
      3. YARN-6093-git08dc09581230ba595ce48fe7d3bc4eb2b6f98091.v4.patch
        10 kB
        Subru Krishnan
      4. YARN-6093-YARN-2915.v1.patch
        10 kB
        Botong Huang
      5. YARN-6093-YARN-2915.v2.patch
        10 kB
        Botong Huang
      6. YARN-6093-YARN-2915.v3.patch
        10 kB
        Botong Huang
      7. YARN-6093-YARN-2915.v4.patch
        10 kB
        Botong Huang
      8. YARN-6093-YARN-2915.v5.patch
        11 kB
        Botong Huang

        Issue Links

          Activity

          Hide
          hadoopqa Hadoop QA added a comment -
          -1 overall



          Vote Subsystem Runtime Comment
          0 reexec 0m 0s Docker mode activated.
          -1 patch 0m 4s YARN-6093 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help.



          Subsystem Report/Notes
          JIRA Issue YARN-6093
          JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12847383/YARN-6093.v1.patch
          Console output https://builds.apache.org/job/PreCommit-YARN-Build/14648/console
          Powered by Apache Yetus 0.5.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 0s Docker mode activated. -1 patch 0m 4s YARN-6093 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help. Subsystem Report/Notes JIRA Issue YARN-6093 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12847383/YARN-6093.v1.patch Console output https://builds.apache.org/job/PreCommit-YARN-Build/14648/console Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
          Hide
          hadoopqa Hadoop QA added a comment -
          +1 overall



          Vote Subsystem Runtime Comment
          0 reexec 0m 13s Docker mode activated.
          +1 @author 0m 0s The patch does not contain any @author tags.
          +1 test4tests 0m 0s The patch appears to include 1 new or modified test files.
          0 mvndep 0m 9s Maven dependency ordering for branch
          +1 mvninstall 24m 22s YARN-2915 passed
          +1 compile 5m 11s YARN-2915 passed
          +1 checkstyle 0m 48s YARN-2915 passed
          +1 mvnsite 1m 1s YARN-2915 passed
          +1 mvneclipse 0m 43s YARN-2915 passed
          +1 findbugs 1m 44s YARN-2915 passed
          +1 javadoc 0m 44s YARN-2915 passed
          0 mvndep 0m 10s Maven dependency ordering for patch
          +1 mvninstall 0m 54s the patch passed
          +1 compile 4m 41s the patch passed
          +1 javac 4m 41s the patch passed
          -0 checkstyle 0m 43s hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2)
          +1 mvnsite 0m 54s the patch passed
          +1 mvneclipse 0m 34s the patch passed
          +1 whitespace 0m 0s The patch has no whitespace issues.
          +1 findbugs 1m 45s the patch passed
          +1 javadoc 0m 44s the patch passed
          +1 unit 1m 35s hadoop-yarn-server-common in the patch passed.
          +1 unit 16m 44s hadoop-yarn-client in the patch passed.
          +1 asflicense 0m 35s The patch does not generate ASF License warnings.
          72m 4s



          Subsystem Report/Notes
          Docker Image:yetus/hadoop:a9ad5d6
          JIRA Issue YARN-6093
          JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12847923/YARN-6093-YARN-2915.v2.patch
          Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
          uname Linux 579254d91e05 3.13.0-103-generic #150-Ubuntu SMP Thu Nov 24 10:34:17 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
          Build tool maven
          Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
          git revision YARN-2915 / c58725b
          Default Java 1.8.0_111
          findbugs v3.0.0
          checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14682/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn.txt
          Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14682/testReport/
          modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn
          Console output https://builds.apache.org/job/PreCommit-YARN-Build/14682/console
          Powered by Apache Yetus 0.5.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 13s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 1 new or modified test files. 0 mvndep 0m 9s Maven dependency ordering for branch +1 mvninstall 24m 22s YARN-2915 passed +1 compile 5m 11s YARN-2915 passed +1 checkstyle 0m 48s YARN-2915 passed +1 mvnsite 1m 1s YARN-2915 passed +1 mvneclipse 0m 43s YARN-2915 passed +1 findbugs 1m 44s YARN-2915 passed +1 javadoc 0m 44s YARN-2915 passed 0 mvndep 0m 10s Maven dependency ordering for patch +1 mvninstall 0m 54s the patch passed +1 compile 4m 41s the patch passed +1 javac 4m 41s the patch passed -0 checkstyle 0m 43s hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2) +1 mvnsite 0m 54s the patch passed +1 mvneclipse 0m 34s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 1m 45s the patch passed +1 javadoc 0m 44s the patch passed +1 unit 1m 35s hadoop-yarn-server-common in the patch passed. +1 unit 16m 44s hadoop-yarn-client in the patch passed. +1 asflicense 0m 35s The patch does not generate ASF License warnings. 72m 4s Subsystem Report/Notes Docker Image:yetus/hadoop:a9ad5d6 JIRA Issue YARN-6093 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12847923/YARN-6093-YARN-2915.v2.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 579254d91e05 3.13.0-103-generic #150-Ubuntu SMP Thu Nov 24 10:34:17 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision YARN-2915 / c58725b Default Java 1.8.0_111 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14682/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14682/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/14682/console Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
          Hide
          subru Subru Krishnan added a comment -

          Thanks Botong Huang for the patch. At high level, it looks good but it'll be great if we can test it e2e in a cluster as this is a nuanced issue.

          Also the checkstyle issue seems fairly trivial to fix.

          Show
          subru Subru Krishnan added a comment - Thanks Botong Huang for the patch. At high level, it looks good but it'll be great if we can test it e2e in a cluster as this is a nuanced issue. Also the checkstyle issue seems fairly trivial to fix.
          Hide
          botong Botong Huang added a comment -

          Thanks Subru Krishnan for the review. Yeah we've tested it e2e in a cluster for about a week now. The fix looks good.

          Show
          botong Botong Huang added a comment - Thanks Subru Krishnan for the review. Yeah we've tested it e2e in a cluster for about a week now. The fix looks good.
          Hide
          subru Subru Krishnan added a comment -

          Renaming patch to allow Yetus to handle force branch update (test).

          Show
          subru Subru Krishnan added a comment - Renaming patch to allow Yetus to handle force branch update (test).
          Hide
          hadoopqa Hadoop QA added a comment -
          -1 overall



          Vote Subsystem Runtime Comment
          0 reexec 0m 0s Docker mode activated.
          -1 patch 0m 4s YARN-6093 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help.



          Subsystem Report/Notes
          JIRA Issue YARN-6093
          JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12850093/YARN-6093-08dc09581230ba595ce48fe7d3bc4eb2b6f98091.v4.patch
          Console output https://builds.apache.org/job/PreCommit-YARN-Build/14793/console
          Powered by Apache Yetus 0.5.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 0s Docker mode activated. -1 patch 0m 4s YARN-6093 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help. Subsystem Report/Notes JIRA Issue YARN-6093 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12850093/YARN-6093-08dc09581230ba595ce48fe7d3bc4eb2b6f98091.v4.patch Console output https://builds.apache.org/job/PreCommit-YARN-Build/14793/console Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
          Hide
          subru Subru Krishnan added a comment -

          Trying again with a correct (hopefully) file name pattern based on: https://github.com/apache/yetus/blob/master/precommit/test-patch.sh#L1356-L1368

          Show
          subru Subru Krishnan added a comment - Trying again with a correct (hopefully) file name pattern based on: https://github.com/apache/yetus/blob/master/precommit/test-patch.sh#L1356-L1368
          Hide
          hadoopqa Hadoop QA added a comment -
          -1 overall



          Vote Subsystem Runtime Comment
          0 reexec 0m 27s Docker mode activated.
          +1 @author 0m 0s The patch does not contain any @author tags.
          +1 test4tests 0m 0s The patch appears to include 1 new or modified test files.
          0 mvndep 0m 26s Maven dependency ordering for branch
          +1 mvninstall 20m 9s YARN-2915 passed
          +1 compile 8m 28s YARN-2915 passed
          +1 checkstyle 0m 55s YARN-2915 passed
          +1 mvnsite 0m 57s YARN-2915 passed
          +1 mvneclipse 0m 44s YARN-2915 passed
          +1 findbugs 1m 33s YARN-2915 passed
          +1 javadoc 0m 44s YARN-2915 passed
          0 mvndep 0m 10s Maven dependency ordering for patch
          +1 mvninstall 0m 39s the patch passed
          +1 compile 6m 3s the patch passed
          +1 javac 6m 3s the patch passed
          +1 checkstyle 0m 43s the patch passed
          +1 mvnsite 0m 53s the patch passed
          +1 mvneclipse 0m 36s the patch passed
          +1 whitespace 0m 0s The patch has no whitespace issues.
          +1 findbugs 1m 35s the patch passed
          +1 javadoc 0m 41s the patch passed
          -1 unit 1m 17s hadoop-yarn-server-common in the patch failed.
          +1 unit 17m 26s hadoop-yarn-client in the patch passed.
          +1 asflicense 0m 33s The patch does not generate ASF License warnings.
          73m 32s



          Reason Tests
          Failed junit tests hadoop.yarn.server.federation.policies.router.TestWeightedRandomRouterPolicy



          Subsystem Report/Notes
          Docker Image:yetus/hadoop:a9ad5d6
          JIRA Issue YARN-6093
          JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12852692/YARN-6093-YARN-2915.v4.patch
          Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
          uname Linux d555cfc3f872 3.13.0-105-generic #152-Ubuntu SMP Fri Dec 2 15:37:11 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
          Build tool maven
          Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
          git revision YARN-2915 / 08dc095
          Default Java 1.8.0_121
          findbugs v3.0.0
          unit https://builds.apache.org/job/PreCommit-YARN-Build/14949/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-common.txt
          Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14949/testReport/
          modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn
          Console output https://builds.apache.org/job/PreCommit-YARN-Build/14949/console
          Powered by Apache Yetus 0.5.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 27s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 1 new or modified test files. 0 mvndep 0m 26s Maven dependency ordering for branch +1 mvninstall 20m 9s YARN-2915 passed +1 compile 8m 28s YARN-2915 passed +1 checkstyle 0m 55s YARN-2915 passed +1 mvnsite 0m 57s YARN-2915 passed +1 mvneclipse 0m 44s YARN-2915 passed +1 findbugs 1m 33s YARN-2915 passed +1 javadoc 0m 44s YARN-2915 passed 0 mvndep 0m 10s Maven dependency ordering for patch +1 mvninstall 0m 39s the patch passed +1 compile 6m 3s the patch passed +1 javac 6m 3s the patch passed +1 checkstyle 0m 43s the patch passed +1 mvnsite 0m 53s the patch passed +1 mvneclipse 0m 36s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 1m 35s the patch passed +1 javadoc 0m 41s the patch passed -1 unit 1m 17s hadoop-yarn-server-common in the patch failed. +1 unit 17m 26s hadoop-yarn-client in the patch passed. +1 asflicense 0m 33s The patch does not generate ASF License warnings. 73m 32s Reason Tests Failed junit tests hadoop.yarn.server.federation.policies.router.TestWeightedRandomRouterPolicy Subsystem Report/Notes Docker Image:yetus/hadoop:a9ad5d6 JIRA Issue YARN-6093 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12852692/YARN-6093-YARN-2915.v4.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux d555cfc3f872 3.13.0-105-generic #152-Ubuntu SMP Fri Dec 2 15:37:11 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision YARN-2915 / 08dc095 Default Java 1.8.0_121 findbugs v3.0.0 unit https://builds.apache.org/job/PreCommit-YARN-Build/14949/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-common.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14949/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/14949/console Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
          Hide
          botong Botong Huang added a comment -

          +Carlo Curino: can you please take a look at the test failure (TestWeightedRandomRouterPolicy.testClusterChosenWithRightProbability) in the last Yetus run? Thanks!

          Show
          botong Botong Huang added a comment - + Carlo Curino : can you please take a look at the test failure (TestWeightedRandomRouterPolicy.testClusterChosenWithRightProbability) in the last Yetus run? Thanks!
          Hide
          botong Botong Huang added a comment - - edited

          Talked to Carlo Curino offline, this is a randomized test, and sometimes it fails because the threshold is too tight. This is irrelevant to the patch here.

          Show
          botong Botong Huang added a comment - - edited Talked to Carlo Curino offline, this is a randomized test, and sometimes it fails because the threshold is too tight. This is irrelevant to the patch here.
          Hide
          subru Subru Krishnan added a comment -

          Botong Huang, can you open a JIRA to track the test failure as we need to fix it.

          Jian He, can you take a quick look at the patch before I commit as this addresses one of your early requests, i.e. not to save the tokens in FederationRMFailoverProxyProvider. Thanks.

          Show
          subru Subru Krishnan added a comment - Botong Huang , can you open a JIRA to track the test failure as we need to fix it. Jian He , can you take a quick look at the patch before I commit as this addresses one of your early requests, i.e. not to save the tokens in FederationRMFailoverProxyProvider . Thanks.
          Hide
          botong Botong Huang added a comment -

          Sure, YARN-6203 created and assigned to Carlo Curino

          Show
          botong Botong Huang added a comment - Sure, YARN-6203 created and assigned to Carlo Curino
          Hide
          botong Botong Huang added a comment -

          v5 patch uploaded. Fixed another issue: when performFailover fails, should fall back to the old proxy instance, rather than return null, causing NullPointerException later.

          Show
          botong Botong Huang added a comment - v5 patch uploaded. Fixed another issue: when performFailover fails, should fall back to the old proxy instance, rather than return null, causing NullPointerException later.
          Hide
          jianhe Jian He added a comment -
          Show
          jianhe Jian He added a comment - lgtm, thanks Botong Huang , Subru Krishnan
          Hide
          hadoopqa Hadoop QA added a comment -
          +1 overall



          Vote Subsystem Runtime Comment
          0 reexec 0m 26s Docker mode activated.
          +1 @author 0m 0s The patch does not contain any @author tags.
          +1 test4tests 0m 0s The patch appears to include 1 new or modified test files.
          0 mvndep 0m 12s Maven dependency ordering for branch
          +1 mvninstall 17m 52s YARN-2915 passed
          +1 compile 8m 48s YARN-2915 passed
          +1 checkstyle 0m 46s YARN-2915 passed
          +1 mvnsite 0m 57s YARN-2915 passed
          +1 mvneclipse 0m 40s YARN-2915 passed
          +1 findbugs 1m 35s YARN-2915 passed
          +1 javadoc 0m 51s YARN-2915 passed
          0 mvndep 0m 10s Maven dependency ordering for patch
          +1 mvninstall 0m 42s the patch passed
          +1 compile 6m 57s the patch passed
          +1 javac 6m 57s the patch passed
          +1 checkstyle 0m 43s the patch passed
          +1 mvnsite 0m 54s the patch passed
          +1 mvneclipse 0m 35s the patch passed
          +1 whitespace 0m 0s The patch has no whitespace issues.
          +1 findbugs 1m 36s the patch passed
          +1 javadoc 0m 41s the patch passed
          +1 unit 1m 18s hadoop-yarn-server-common in the patch passed.
          +1 unit 17m 54s hadoop-yarn-client in the patch passed.
          +1 asflicense 0m 38s The patch does not generate ASF License warnings.
          72m 48s



          Subsystem Report/Notes
          Docker Image:yetus/hadoop:a9ad5d6
          JIRA Issue YARN-6093
          JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12853838/YARN-6093-YARN-2915.v5.patch
          Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
          uname Linux 2127028b7d56 3.13.0-108-generic #155-Ubuntu SMP Wed Jan 11 16:58:52 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
          Build tool maven
          Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
          git revision YARN-2915 / 8d9be84
          Default Java 1.8.0_121
          findbugs v3.0.0
          Test Results https://builds.apache.org/job/PreCommit-YARN-Build/15032/testReport/
          modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn
          Console output https://builds.apache.org/job/PreCommit-YARN-Build/15032/console
          Powered by Apache Yetus 0.5.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 26s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 1 new or modified test files. 0 mvndep 0m 12s Maven dependency ordering for branch +1 mvninstall 17m 52s YARN-2915 passed +1 compile 8m 48s YARN-2915 passed +1 checkstyle 0m 46s YARN-2915 passed +1 mvnsite 0m 57s YARN-2915 passed +1 mvneclipse 0m 40s YARN-2915 passed +1 findbugs 1m 35s YARN-2915 passed +1 javadoc 0m 51s YARN-2915 passed 0 mvndep 0m 10s Maven dependency ordering for patch +1 mvninstall 0m 42s the patch passed +1 compile 6m 57s the patch passed +1 javac 6m 57s the patch passed +1 checkstyle 0m 43s the patch passed +1 mvnsite 0m 54s the patch passed +1 mvneclipse 0m 35s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 1m 36s the patch passed +1 javadoc 0m 41s the patch passed +1 unit 1m 18s hadoop-yarn-server-common in the patch passed. +1 unit 17m 54s hadoop-yarn-client in the patch passed. +1 asflicense 0m 38s The patch does not generate ASF License warnings. 72m 48s Subsystem Report/Notes Docker Image:yetus/hadoop:a9ad5d6 JIRA Issue YARN-6093 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12853838/YARN-6093-YARN-2915.v5.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 2127028b7d56 3.13.0-108-generic #155-Ubuntu SMP Wed Jan 11 16:58:52 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision YARN-2915 / 8d9be84 Default Java 1.8.0_121 findbugs v3.0.0 Test Results https://builds.apache.org/job/PreCommit-YARN-Build/15032/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/15032/console Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
          Hide
          subru Subru Krishnan added a comment -

          Thanks Botong Huang for the patch and Jian He for the validation, I just committed this to branch YARN-2915.

          Show
          subru Subru Krishnan added a comment - Thanks Botong Huang for the patch and Jian He for the validation, I just committed this to branch YARN-2915 .

            People

            • Assignee:
              botong Botong Huang
              Reporter:
              botong Botong Huang
            • Votes:
              0 Vote for this issue
              Watchers:
              10 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development