Details

    • Target Version/s:
    • Hadoop Flags:
      Reviewed

      Description

      In 2.6.0, we added an option called "-directlyAccessNodeLabelStore" to make RM can start with label-configured queue settings. After YARN-2918, we don't need this option any more, admin can configure queue setting, start RM and configure node label via RMAdminCLI without any error.

      In addition, this option is very restrictive, first it needs to run on the same node where RM is running if admin configured to store labels in local disk.

      Second, when admin run the option when RM is running, multiple process write to a same file can happen, this could make node label store becomes invalid.

      1. YARN-3581.20150525-1.patch
        7 kB
        Naganarasimha G R
      2. YARN-3581.20150528-1.patch
        9 kB
        Naganarasimha G R

        Activity

        Hide
        aw Allen Wittenauer added a comment -

        Removing a command line option is an incompatible change.

        Show
        aw Allen Wittenauer added a comment - Removing a command line option is an incompatible change.
        Hide
        leftnoteasy Wangda Tan added a comment -

        Thanks for reminding, Allen Wittenauer. Updated title to deprecate this option.

        Show
        leftnoteasy Wangda Tan added a comment - Thanks for reminding, Allen Wittenauer . Updated title to deprecate this option.
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Hi Tan, Wangda
        There was one trivial issue with the CLI, can you take care of that as part of this patch:
        we are mentioning in help for yarn rmadmin
        -addToClusterNodeLabels [label1(exclusive=true),label2(exclusive=false),label3]
        but its not captured that : if not specified then default exclusive=true

        Show
        Naganarasimha Naganarasimha G R added a comment - Hi Tan, Wangda There was one trivial issue with the CLI, can you take care of that as part of this patch: we are mentioning in help for yarn rmadmin -addToClusterNodeLabels [label1(exclusive=true),label2(exclusive=false),label3] but its not captured that : if not specified then default exclusive=true
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Hi Tan, Wangda
        Found few other problems too

        1. In help for yarn rmadmin its displayed as -addToClusterNodeLabels [label1(exclusive=true),label2(exclusive=false),label3] but its not captured that : if not specified then default exclusive=true.
        2. while adding non exclusive label ./yarn rmadmin -addToClusterNodeLabels label1(exclusive=false) got error as bash: syntax error near unexpected token `('. and succeeds when ./yarn rmadmin -addToClusterNodeLabels label2\\(exclusive=false
          )
          i.e. when we escape the bracket, hence was wondering can we support it as yarn rmadmin -addToClusterNodeLabels [label1|exclusive=true,label2|exclusive=false,label3] or yarn rmadmin -addToClusterNodeLabels [label1|exclusive,label2|nonexclusive,label3]
        3. If no options are passed for -addToClusterNodeLabels,-removeFromClusterNodeLabels,-replaceLabelsOnNode then we get error message as No cluster node-labels are specified instead it should error msg should be followed with the command usage
        4. -directlyAccessNodeLabelStore is present in yarn rmadmin and {{ yarn cluster}} and both places we need to make deprecated.

        If you are ok i can take this issue up and will also fix the ones pointed out by me.

        Show
        Naganarasimha Naganarasimha G R added a comment - Hi Tan, Wangda Found few other problems too In help for yarn rmadmin its displayed as -addToClusterNodeLabels [label1(exclusive=true),label2(exclusive=false),label3] but its not captured that : if not specified then default exclusive=true. while adding non exclusive label ./yarn rmadmin -addToClusterNodeLabels label1(exclusive=false) got error as bash: syntax error near unexpected token `(' . and succeeds when ./yarn rmadmin -addToClusterNodeLabels label2\\(exclusive=false ) i.e. when we escape the bracket, hence was wondering can we support it as yarn rmadmin -addToClusterNodeLabels [label1|exclusive=true,label2|exclusive=false,label3] or yarn rmadmin -addToClusterNodeLabels [label1|exclusive,label2|nonexclusive,label3] If no options are passed for -addToClusterNodeLabels , -removeFromClusterNodeLabels , -replaceLabelsOnNode then we get error message as No cluster node-labels are specified instead it should error msg should be followed with the command usage -directlyAccessNodeLabelStore is present in yarn rmadmin and {{ yarn cluster}} and both places we need to make deprecated. If you are ok i can take this issue up and will also fix the ones pointed out by me.
        Hide
        leftnoteasy Wangda Tan added a comment -

        Hi Naganarasimha G R,
        Thanks for trying and thinking about this, assigning to you, please go ahead.

        #1/#3/#4 make sense to me

        For #2, good point, I think we need add ".." for label(...),label(...), like "label(...),label(...)". This should in usage/help message as well. I'm open to change "( )" to "[ ]", but I'm not sure if other shell requires escape "[..]" too. For now I prefer to keep "(...)", not adding new syntax, but document it in help message.

        In addition, similar to #2, "-replaceLabelsOnNode" has problem when admin specified multiple "host=" in a same line without using "..". I think we should add using "..." in help message as well. If admin forgot to add "...", we should handle it instead of ignore "host=" after first one.

        Let me know your thoughts.

        Thanks,
        Wangda

        Show
        leftnoteasy Wangda Tan added a comment - Hi Naganarasimha G R , Thanks for trying and thinking about this, assigning to you, please go ahead. #1/#3/#4 make sense to me For #2, good point, I think we need add ".." for label(...),label(...) , like "label(...),label(...)" . This should in usage/help message as well. I'm open to change "( )" to "[ ]", but I'm not sure if other shell requires escape " [..] " too. For now I prefer to keep "(...)", not adding new syntax, but document it in help message. In addition, similar to #2, "-replaceLabelsOnNode" has problem when admin specified multiple "host=" in a same line without using "..". I think we should add using "..." in help message as well. If admin forgot to add "...", we should handle it instead of ignore "host=" after first one. Let me know your thoughts. Thanks, Wangda
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Hi Tan, Wangda,
        I have corrected the issues mentioned earlier and also added one more fix:
        "[ ..]" square brackets are used when the parameter is optional, hence converted them to "<..>" as arguments are required in case of labels modification.
        Please inform if you require a separate patch for 7.1 branch (1,3,4 & the one mentioned above).

        Show
        Naganarasimha Naganarasimha G R added a comment - Hi Tan, Wangda , I have corrected the issues mentioned earlier and also added one more fix: "[ ..]" square brackets are used when the parameter is optional, hence converted them to "<..>" as arguments are required in case of labels modification. Please inform if you require a separate patch for 7.1 branch (1,3,4 & the one mentioned above).
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        missed to add changes for ClusterCLI...

        Show
        Naganarasimha Naganarasimha G R added a comment - missed to add changes for ClusterCLI...
        Hide
        hadoopqa Hadoop QA added a comment -



        -1 overall



        Vote Subsystem Runtime Comment
        0 pre-patch 14m 36s Pre-patch trunk compilation is healthy.
        +1 @author 0m 0s The patch does not contain any @author tags.
        -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
        +1 javac 7m 31s There were no new javac warning messages.
        +1 javadoc 9m 34s There were no new javadoc warning messages.
        +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings.
        -1 checkstyle 0m 30s The applied patch generated 1 new checkstyle issues (total was 37, now 38).
        +1 whitespace 0m 0s The patch has no lines that end in whitespace.
        +1 install 1m 33s mvn install still works.
        +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse.
        +1 findbugs 0m 44s The patch does not introduce any new Findbugs (version 3.0.0) warnings.
        +1 yarn tests 6m 58s Tests passed in hadoop-yarn-client.
            42m 26s  



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12735192/YARN-3581.20150525-1.patch
        Optional Tests javadoc javac unit findbugs checkstyle
        git revision trunk / ada233b
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/8073/artifact/patchprocess/diffcheckstylehadoop-yarn-client.txt
        hadoop-yarn-client test log https://builds.apache.org/job/PreCommit-YARN-Build/8073/artifact/patchprocess/testrun_hadoop-yarn-client.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8073/testReport/
        Java 1.7.0_55
        uname Linux asf906.gq1.ygridcore.net 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
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/8073/console

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 36s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac 7m 31s There were no new javac warning messages. +1 javadoc 9m 34s There were no new javadoc warning messages. +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings. -1 checkstyle 0m 30s The applied patch generated 1 new checkstyle issues (total was 37, now 38). +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 install 1m 33s mvn install still works. +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse. +1 findbugs 0m 44s The patch does not introduce any new Findbugs (version 3.0.0) warnings. +1 yarn tests 6m 58s Tests passed in hadoop-yarn-client.     42m 26s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12735192/YARN-3581.20150525-1.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / ada233b checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/8073/artifact/patchprocess/diffcheckstylehadoop-yarn-client.txt hadoop-yarn-client test log https://builds.apache.org/job/PreCommit-YARN-Build/8073/artifact/patchprocess/testrun_hadoop-yarn-client.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8073/testReport/ Java 1.7.0_55 uname Linux asf906.gq1.ygridcore.net 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 Console output https://builds.apache.org/job/PreCommit-YARN-Build/8073/console This message was automatically generated.
        Hide
        hadoopqa Hadoop QA added a comment -



        -1 overall



        Vote Subsystem Runtime Comment
        0 pre-patch 14m 29s Pre-patch trunk compilation is healthy.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 tests included 0m 0s The patch appears to include 1 new or modified test files.
        +1 javac 7m 31s There were no new javac warning messages.
        +1 javadoc 9m 36s There were no new javadoc warning messages.
        +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings.
        -1 checkstyle 0m 28s The applied patch generated 2 new checkstyle issues (total was 40, now 42).
        +1 whitespace 0m 1s The patch has no lines that end in whitespace.
        +1 install 1m 31s mvn install still works.
        +1 eclipse:eclipse 0m 38s The patch built with eclipse:eclipse.
        +1 findbugs 0m 43s The patch does not introduce any new Findbugs (version 3.0.0) warnings.
        +1 yarn tests 6m 53s Tests passed in hadoop-yarn-client.
            42m 15s  



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12735195/YARN-3581.20150525-1.patch
        Optional Tests javadoc javac unit findbugs checkstyle
        git revision trunk / ada233b
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/8074/artifact/patchprocess/diffcheckstylehadoop-yarn-client.txt
        hadoop-yarn-client test log https://builds.apache.org/job/PreCommit-YARN-Build/8074/artifact/patchprocess/testrun_hadoop-yarn-client.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8074/testReport/
        Java 1.7.0_55
        uname Linux asf909.gq1.ygridcore.net 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
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/8074/console

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 29s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 1 new or modified test files. +1 javac 7m 31s There were no new javac warning messages. +1 javadoc 9m 36s There were no new javadoc warning messages. +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings. -1 checkstyle 0m 28s The applied patch generated 2 new checkstyle issues (total was 40, now 42). +1 whitespace 0m 1s The patch has no lines that end in whitespace. +1 install 1m 31s mvn install still works. +1 eclipse:eclipse 0m 38s The patch built with eclipse:eclipse. +1 findbugs 0m 43s The patch does not introduce any new Findbugs (version 3.0.0) warnings. +1 yarn tests 6m 53s Tests passed in hadoop-yarn-client.     42m 15s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12735195/YARN-3581.20150525-1.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / ada233b checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/8074/artifact/patchprocess/diffcheckstylehadoop-yarn-client.txt hadoop-yarn-client test log https://builds.apache.org/job/PreCommit-YARN-Build/8074/artifact/patchprocess/testrun_hadoop-yarn-client.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8074/testReport/ Java 1.7.0_55 uname Linux asf909.gq1.ygridcore.net 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 Console output https://builds.apache.org/job/PreCommit-YARN-Build/8074/console This message was automatically generated.
        Hide
        leftnoteasy Wangda Tan added a comment -

        Naganarasimha G R. Thanks for working on this, some comments:

        • (Deprecated! Support will be removed in future) Directly access node label store, }}, is it better to make it: {{(This is DEPRECATED, will be removed in future releases)...?
        • RMAdminCLI put the "deprecated.." message in args option instead of "help".
        • printHelp in RMAdminCLI should be consistency with usage? For changes of <...>
        Show
        leftnoteasy Wangda Tan added a comment - Naganarasimha G R . Thanks for working on this, some comments: (Deprecated! Support will be removed in future) Directly access node label store, }}, is it better to make it: {{(This is DEPRECATED, will be removed in future releases)... ? RMAdminCLI put the "deprecated.." message in args option instead of "help". printHelp in RMAdminCLI should be consistency with usage? For changes of <...>
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        hi Tan, Wangda,

        Regarding deprecated message as argument, -removeFromClusterNodeLabels had a comment that (label splitted by ","), so thought imp info can be shown in this way. will move it to description . but one more thing, shall i totally remove description and have only this Deprecated message, so that no one will use it ?
        others will get it corrected.

        Show
        Naganarasimha Naganarasimha G R added a comment - hi Tan, Wangda , Regarding deprecated message as argument, -removeFromClusterNodeLabels had a comment that (label splitted by ",") , so thought imp info can be shown in this way. will move it to description . but one more thing, shall i totally remove description and have only this Deprecated message, so that no one will use it ? others will get it corrected.
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Hi Tan, Wangda, updated a patch with your earlier review comments fixed but please check my previous comment and also confirm if you require patch for 2.7.1 branch (as per offline discussion you wanted fix in 2.7.1, so that it will be used and later deleted in 2.8)?

        Show
        Naganarasimha Naganarasimha G R added a comment - Hi Tan, Wangda , updated a patch with your earlier review comments fixed but please check my previous comment and also confirm if you require patch for 2.7.1 branch (as per offline discussion you wanted fix in 2.7.1, so that it will be used and later deleted in 2.8)?
        Hide
        leftnoteasy Wangda Tan added a comment -

        Naganarasimha G R,
        Thanks for update, the latest patch looks good. And I think it's better to add to 2.7.1 as well to avoid people use it. We will not remove these options in 2.8, but we should let people know about the risk.

        Wangda

        Show
        leftnoteasy Wangda Tan added a comment - Naganarasimha G R , Thanks for update, the latest patch looks good. And I think it's better to add to 2.7.1 as well to avoid people use it. We will not remove these options in 2.8, but we should let people know about the risk. Wangda
        Hide
        hadoopqa Hadoop QA added a comment -



        -1 overall



        Vote Subsystem Runtime Comment
        0 pre-patch 14m 39s Pre-patch trunk compilation is healthy.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 tests included 0m 0s The patch appears to include 2 new or modified test files.
        +1 javac 7m 37s There were no new javac warning messages.
        +1 javadoc 9m 36s There were no new javadoc warning messages.
        +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings.
        -1 checkstyle 0m 29s The applied patch generated 6 new checkstyle issues (total was 40, now 42).
        +1 whitespace 0m 0s The patch has no lines that end in whitespace.
        +1 install 1m 33s mvn install still works.
        +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse.
        +1 findbugs 0m 44s The patch does not introduce any new Findbugs (version 3.0.0) warnings.
        +1 yarn tests 6m 52s Tests passed in hadoop-yarn-client.
            42m 29s  



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12735661/YARN-3581.20150528-1.patch
        Optional Tests javadoc javac unit findbugs checkstyle
        git revision trunk / c46d4ba
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/8102/artifact/patchprocess/diffcheckstylehadoop-yarn-client.txt
        hadoop-yarn-client test log https://builds.apache.org/job/PreCommit-YARN-Build/8102/artifact/patchprocess/testrun_hadoop-yarn-client.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8102/testReport/
        Java 1.7.0_55
        uname Linux asf902.gq1.ygridcore.net 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
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/8102/console

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 39s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 2 new or modified test files. +1 javac 7m 37s There were no new javac warning messages. +1 javadoc 9m 36s There were no new javadoc warning messages. +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings. -1 checkstyle 0m 29s The applied patch generated 6 new checkstyle issues (total was 40, now 42). +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 install 1m 33s mvn install still works. +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse. +1 findbugs 0m 44s The patch does not introduce any new Findbugs (version 3.0.0) warnings. +1 yarn tests 6m 52s Tests passed in hadoop-yarn-client.     42m 29s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12735661/YARN-3581.20150528-1.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / c46d4ba checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/8102/artifact/patchprocess/diffcheckstylehadoop-yarn-client.txt hadoop-yarn-client test log https://builds.apache.org/job/PreCommit-YARN-Build/8102/artifact/patchprocess/testrun_hadoop-yarn-client.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8102/testReport/ Java 1.7.0_55 uname Linux asf902.gq1.ygridcore.net 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 Console output https://builds.apache.org/job/PreCommit-YARN-Build/8102/console This message was automatically generated.
        Hide
        leftnoteasy Wangda Tan added a comment -

        Think about this, this is not a critical issue for 2.7.1, so I think it's better not to commit to 2.7.1. Removed it from target version. Committing.

        Show
        leftnoteasy Wangda Tan added a comment - Think about this, this is not a critical issue for 2.7.1, so I think it's better not to commit to 2.7.1. Removed it from target version. Committing.
        Hide
        leftnoteasy Wangda Tan added a comment -

        Committed to branch-2/trunk. Thanks Naganarasimha G R!

        Show
        leftnoteasy Wangda Tan added a comment - Committed to branch-2/trunk. Thanks Naganarasimha G R !
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Hadoop-trunk-Commit #7910 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7910/)
        YARN-3581. Deprecate -directlyAccessNodeLabelStore in RMAdminCLI. (Naganarasimha G R via wangda) (wangda: rev cab7674e54c4fe56838668462de99a6787841309)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/RMAdminCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestRMAdminCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestClusterCLI.java
        • hadoop-yarn-project/CHANGES.txt
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ClusterCLI.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-trunk-Commit #7910 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7910/ ) YARN-3581 . Deprecate -directlyAccessNodeLabelStore in RMAdminCLI. (Naganarasimha G R via wangda) (wangda: rev cab7674e54c4fe56838668462de99a6787841309) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/RMAdminCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestRMAdminCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestClusterCLI.java hadoop-yarn-project/CHANGES.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ClusterCLI.java
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Thanks Tan, Wangda for reviewing and committing the patch.

        Show
        Naganarasimha Naganarasimha G R added a comment - Thanks Tan, Wangda for reviewing and committing the patch.
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #211 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/211/)
        YARN-3581. Deprecate -directlyAccessNodeLabelStore in RMAdminCLI. (Naganarasimha G R via wangda) (wangda: rev cab7674e54c4fe56838668462de99a6787841309)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/RMAdminCLI.java
        • hadoop-yarn-project/CHANGES.txt
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ClusterCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestRMAdminCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestClusterCLI.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #211 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/211/ ) YARN-3581 . Deprecate -directlyAccessNodeLabelStore in RMAdminCLI. (Naganarasimha G R via wangda) (wangda: rev cab7674e54c4fe56838668462de99a6787841309) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/RMAdminCLI.java hadoop-yarn-project/CHANGES.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ClusterCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestRMAdminCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestClusterCLI.java
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Hadoop-Yarn-trunk #941 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/941/)
        YARN-3581. Deprecate -directlyAccessNodeLabelStore in RMAdminCLI. (Naganarasimha G R via wangda) (wangda: rev cab7674e54c4fe56838668462de99a6787841309)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ClusterCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestRMAdminCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestClusterCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/RMAdminCLI.java
        • hadoop-yarn-project/CHANGES.txt
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Yarn-trunk #941 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/941/ ) YARN-3581 . Deprecate -directlyAccessNodeLabelStore in RMAdminCLI. (Naganarasimha G R via wangda) (wangda: rev cab7674e54c4fe56838668462de99a6787841309) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ClusterCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestRMAdminCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestClusterCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/RMAdminCLI.java hadoop-yarn-project/CHANGES.txt
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Hdfs-trunk #2139 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2139/)
        YARN-3581. Deprecate -directlyAccessNodeLabelStore in RMAdminCLI. (Naganarasimha G R via wangda) (wangda: rev cab7674e54c4fe56838668462de99a6787841309)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestClusterCLI.java
        • hadoop-yarn-project/CHANGES.txt
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestRMAdminCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ClusterCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/RMAdminCLI.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk #2139 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2139/ ) YARN-3581 . Deprecate -directlyAccessNodeLabelStore in RMAdminCLI. (Naganarasimha G R via wangda) (wangda: rev cab7674e54c4fe56838668462de99a6787841309) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestClusterCLI.java hadoop-yarn-project/CHANGES.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestRMAdminCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ClusterCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/RMAdminCLI.java
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #199 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/199/)
        YARN-3581. Deprecate -directlyAccessNodeLabelStore in RMAdminCLI. (Naganarasimha G R via wangda) (wangda: rev cab7674e54c4fe56838668462de99a6787841309)

        • hadoop-yarn-project/CHANGES.txt
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ClusterCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestClusterCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestRMAdminCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/RMAdminCLI.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #199 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/199/ ) YARN-3581 . Deprecate -directlyAccessNodeLabelStore in RMAdminCLI. (Naganarasimha G R via wangda) (wangda: rev cab7674e54c4fe56838668462de99a6787841309) hadoop-yarn-project/CHANGES.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ClusterCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestClusterCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestRMAdminCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/RMAdminCLI.java
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Hadoop-Mapreduce-trunk-Java8 #209 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/209/)
        YARN-3581. Deprecate -directlyAccessNodeLabelStore in RMAdminCLI. (Naganarasimha G R via wangda) (wangda: rev cab7674e54c4fe56838668462de99a6787841309)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestClusterCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ClusterCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestRMAdminCLI.java
        • hadoop-yarn-project/CHANGES.txt
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/RMAdminCLI.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Mapreduce-trunk-Java8 #209 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/209/ ) YARN-3581 . Deprecate -directlyAccessNodeLabelStore in RMAdminCLI. (Naganarasimha G R via wangda) (wangda: rev cab7674e54c4fe56838668462de99a6787841309) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestClusterCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ClusterCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestRMAdminCLI.java hadoop-yarn-project/CHANGES.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/RMAdminCLI.java
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Hadoop-Mapreduce-trunk #2157 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2157/)
        YARN-3581. Deprecate -directlyAccessNodeLabelStore in RMAdminCLI. (Naganarasimha G R via wangda) (wangda: rev cab7674e54c4fe56838668462de99a6787841309)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestRMAdminCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestClusterCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/RMAdminCLI.java
        • hadoop-yarn-project/CHANGES.txt
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ClusterCLI.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Mapreduce-trunk #2157 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2157/ ) YARN-3581 . Deprecate -directlyAccessNodeLabelStore in RMAdminCLI. (Naganarasimha G R via wangda) (wangda: rev cab7674e54c4fe56838668462de99a6787841309) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestRMAdminCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/cli/TestClusterCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/RMAdminCLI.java hadoop-yarn-project/CHANGES.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ClusterCLI.java

          People

          • Assignee:
            Naganarasimha Naganarasimha G R
            Reporter:
            leftnoteasy Wangda Tan
          • Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development