Hadoop HDFS
  1. Hadoop HDFS
  2. HDFS-2949

HA: Add check to active state transition to prevent operator-induced split brain

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.24.0
    • Fix Version/s: 3.0.0, 2.5.0
    • Component/s: ha, namenode
    • Labels:
      None
    • Target Version/s:
    • Hadoop Flags:
      Reviewed

      Description

      Currently, if the administrator mistakenly calls "-transitionToActive" on one NN while the other one is still active, all hell will break loose. We can add a simple check by having the NN make a getServiceState() RPC to its peer with a short (~1 second?) timeout. If the RPC succeeds and indicates the other node is active, it should refuse to enter active mode. If the RPC fails or indicates standby, it can proceed.

      This is just meant as a preventative safety check - we still expect users to use the "-failover" command which has other checks plus fencing built in.

      1. HDFS-2949.patch
        8 kB
        Rushabh S Shah
      2. HDFS-2949-v2.patch
        8 kB
        Rushabh S Shah
      3. HDFS-2949-v3.patch
        12 kB
        Rushabh S Shah

        Issue Links

          Activity

          Hide
          Hari Mankude added a comment -

          If -failover command can handle this situation and other situations correctly, why not deprecate -transitiontoActive entirely?

          Show
          Hari Mankude added a comment - If -failover command can handle this situation and other situations correctly, why not deprecate -transitiontoActive entirely?
          Hide
          Todd Lipcon added a comment -

          I think we should probably un-document the transitionTo* commands, but leave them as a safety valve. It's nice to have direct access to these RPCs just in case there's some problem with one of the safer methods and you need a workaround without recompiling the client.

          That said, having the safety check described in this JIRA is still valuable, even using haadmin -failover, in case the admin has a messed up configuration in some way (eg the fencing script returns true but did not in fact fence the standby correctly)

          Show
          Todd Lipcon added a comment - I think we should probably un-document the transitionTo* commands, but leave them as a safety valve. It's nice to have direct access to these RPCs just in case there's some problem with one of the safer methods and you need a workaround without recompiling the client. That said, having the safety check described in this JIRA is still valuable, even using haadmin -failover, in case the admin has a messed up configuration in some way (eg the fencing script returns true but did not in fact fence the standby correctly)
          Hide
          Uma Maheswara Rao G added a comment -

          That said, having the safety check described in this JIRA is still valuable,

          Agreed with this point to add safety checks. But anyway this can not solve 100% split barain scenarios right? (ex: small network breakage between active and standby and admin accidentally executed -transitiontoActive on standby.) I think this will be addressed in future as part of Automatic failover and shared storage fencing. But when admins deals directly with command line for some maintanence purpose, this case may occur right?
          Also for the apis transitionTo*, do we need to take the confirmation from the user before actually transitioning? this may give some more attention to the admin for proceeding.

          Show
          Uma Maheswara Rao G added a comment - That said, having the safety check described in this JIRA is still valuable, Agreed with this point to add safety checks. But anyway this can not solve 100% split barain scenarios right? (ex: small network breakage between active and standby and admin accidentally executed -transitiontoActive on standby.) I think this will be addressed in future as part of Automatic failover and shared storage fencing. But when admins deals directly with command line for some maintanence purpose, this case may occur right? Also for the apis transitionTo*, do we need to take the confirmation from the user before actually transitioning? this may give some more attention to the admin for proceeding.
          Hide
          Todd Lipcon added a comment -

          Yep, this is not supposed to solve issues, just to prevent a mistake in the common case. Fencing is the correct answer to prevent split brain in the general case.

          Asking for confirmation might be a nice improvement as well, so long as there's a --force option.

          Show
          Todd Lipcon added a comment - Yep, this is not supposed to solve issues, just to prevent a mistake in the common case. Fencing is the correct answer to prevent split brain in the general case. Asking for confirmation might be a nice improvement as well, so long as there's a --force option.
          Hide
          Aaron T. Myers added a comment -

          Converting to top-level issue with commit of HDFS-1623.

          Show
          Aaron T. Myers added a comment - Converting to top-level issue with commit of HDFS-1623 .
          Hide
          Todd Lipcon added a comment -

          Another safety check here is to make sure that the transaction IDs match between the nodes before going active.

          Show
          Todd Lipcon added a comment - Another safety check here is to make sure that the transaction IDs match between the nodes before going active.
          Hide
          Rushabh S Shah added a comment -

          Added a check to see whether any other namenode is active or not.
          If not active then only transition to Active will go through otherwise not.

          Show
          Rushabh S Shah added a comment - Added a check to see whether any other namenode is active or not. If not active then only transition to Active will go through otherwise not.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12642029/HDFS-2949.patch
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 1 new or modified test files.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          -1 core tests. The patch failed these unit tests in hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs:

          org.apache.hadoop.hdfs.tools.TestDFSHAAdmin

          +1 contrib tests. The patch passed contrib unit tests.

          Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/6744//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/6744//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12642029/HDFS-2949.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 core tests . The patch failed these unit tests in hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.tools.TestDFSHAAdmin +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/6744//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/6744//console This message is automatically generated.
          Hide
          Rushabh S Shah added a comment -

          Broke one test case

          Show
          Rushabh S Shah added a comment - Broke one test case
          Hide
          Rushabh S Shah added a comment -

          Attaching a new patch correcting the test case

          Show
          Rushabh S Shah added a comment - Attaching a new patch correcting the test case
          Hide
          Hadoop QA added a comment -

          +1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12642104/HDFS-2949-v2.patch
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 2 new or modified test files.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          +1 core tests. The patch passed unit tests in hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs.

          +1 contrib tests. The patch passed contrib unit tests.

          Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/6749//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/6749//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - +1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12642104/HDFS-2949-v2.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 2 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs. +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/6749//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/6749//console This message is automatically generated.
          Hide
          Kihwal Lee added a comment -

          In DFSHAAdmin#getTargetIds(), nameserviceId should be used, if available, instead of unconditionally calling DFSUtil.getNamenodeNameServiceId(getConf()).

          When checking the other node, RPC proxy should be created with a short timeout (e.g. 5 seconds). We may simply use rpcTimeoutForChecks and set HA_FC_CLI_CHECK_TIMEOUT_KEY in config to whatever value we want. If getServiceStatus() throws an exception, it should be reported to the user and the requested transition should not happen.

          Also, allow "forceActive" to be specifiable for transitionToActive command. Specifying "forceActive" will force the transition even if the check fails with an exception. If getServiceStatus() is successful and the other node is active, the transition should not be allowed even if "forceActive" is specified.

          Show
          Kihwal Lee added a comment - In DFSHAAdmin#getTargetIds() , nameserviceId should be used, if available, instead of unconditionally calling DFSUtil.getNamenodeNameServiceId(getConf()) . When checking the other node, RPC proxy should be created with a short timeout (e.g. 5 seconds). We may simply use rpcTimeoutForChecks and set HA_FC_CLI_CHECK_TIMEOUT_KEY in config to whatever value we want. If getServiceStatus() throws an exception, it should be reported to the user and the requested transition should not happen. Also, allow "forceActive" to be specifiable for transitionToActive command. Specifying "forceActive" will force the transition even if the check fails with an exception. If getServiceStatus() is successful and the other node is active, the transition should not be allowed even if "forceActive" is specified.
          Hide
          Rushabh S Shah added a comment -

          Thanks Kihwal for your comments.
          Attaching a new patch incorporating all of your comments.

          Show
          Rushabh S Shah added a comment - Thanks Kihwal for your comments. Attaching a new patch incorporating all of your comments.
          Hide
          Kihwal Lee added a comment -

          I've manually kicked precommit.

          Show
          Kihwal Lee added a comment - I've manually kicked precommit.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12644835/HDFS-2949-v3.patch
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 2 new or modified test files.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. There were no new javadoc warning messages.

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          -1 core tests. The patch failed these unit tests in hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs:

          org.apache.hadoop.hdfs.server.balancer.TestBalancerWithNodeGroup

          +1 contrib tests. The patch passed contrib unit tests.

          Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/6899//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/6899//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12644835/HDFS-2949-v3.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 2 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 core tests . The patch failed these unit tests in hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.server.balancer.TestBalancerWithNodeGroup +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/6899//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/6899//console This message is automatically generated.
          Hide
          Kihwal Lee added a comment -

          +1 The patch looks good.

          Show
          Kihwal Lee added a comment - +1 The patch looks good.
          Hide
          Kihwal Lee added a comment -

          I've committed this to branch-2 and trunk. Thanks for working on the fix, Rushabh.

          Show
          Kihwal Lee added a comment - I've committed this to branch-2 and trunk. Thanks for working on the fix, Rushabh.
          Hide
          Rushabh S Shah added a comment -

          Thanks Kihwal for reviewing and committing the patch.

          Show
          Rushabh S Shah added a comment - Thanks Kihwal for reviewing and committing the patch.
          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Hadoop-trunk-Commit #5605 (See https://builds.apache.org/job/Hadoop-trunk-Commit/5605/)
          HDFS-2949. Add check to active state transition to prevent operator-induced split brain. Contributed by Rushabh S Shah. (kihwal: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1594709)

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ha/HAAdmin.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NameNode.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSHAAdmin.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdminMiniCluster.java
          Show
          Hudson added a comment - SUCCESS: Integrated in Hadoop-trunk-Commit #5605 (See https://builds.apache.org/job/Hadoop-trunk-Commit/5605/ ) HDFS-2949 . Add check to active state transition to prevent operator-induced split brain. Contributed by Rushabh S Shah. (kihwal: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1594709 ) /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ha/HAAdmin.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NameNode.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSHAAdmin.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdminMiniCluster.java
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Hadoop-Yarn-trunk #562 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/562/)
          HDFS-2949. Add check to active state transition to prevent operator-induced split brain. Contributed by Rushabh S Shah. (kihwal: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1594709)

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ha/HAAdmin.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NameNode.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSHAAdmin.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdminMiniCluster.java
          Show
          Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk #562 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/562/ ) HDFS-2949 . Add check to active state transition to prevent operator-induced split brain. Contributed by Rushabh S Shah. (kihwal: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1594709 ) /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ha/HAAdmin.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NameNode.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSHAAdmin.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdminMiniCluster.java
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Hadoop-Hdfs-trunk #1754 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1754/)
          HDFS-2949. Add check to active state transition to prevent operator-induced split brain. Contributed by Rushabh S Shah. (kihwal: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1594709)

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ha/HAAdmin.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NameNode.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSHAAdmin.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdminMiniCluster.java
          Show
          Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk #1754 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1754/ ) HDFS-2949 . Add check to active state transition to prevent operator-induced split brain. Contributed by Rushabh S Shah. (kihwal: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1594709 ) /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ha/HAAdmin.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NameNode.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSHAAdmin.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdminMiniCluster.java
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Hadoop-Mapreduce-trunk #1780 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1780/)
          HDFS-2949. Add check to active state transition to prevent operator-induced split brain. Contributed by Rushabh S Shah. (kihwal: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1594709)

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ha/HAAdmin.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NameNode.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSHAAdmin.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java
          • /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdminMiniCluster.java
          Show
          Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk #1780 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1780/ ) HDFS-2949 . Add check to active state transition to prevent operator-induced split brain. Contributed by Rushabh S Shah. (kihwal: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1594709 ) /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ha/HAAdmin.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NameNode.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSHAAdmin.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdmin.java /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSHAAdminMiniCluster.java
          Hide
          Zhijie Shen added a comment -

          This patch seems to break YARN-2075.

          Show
          Zhijie Shen added a comment - This patch seems to break YARN-2075 .

            People

            • Assignee:
              Rushabh S Shah
              Reporter:
              Todd Lipcon
            • Votes:
              0 Vote for this issue
              Watchers:
              10 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development