Hadoop YARN
  1. Hadoop YARN
  2. YARN-9

Rename YARN_HOME to HADOOP_YARN_HOME

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.2-alpha
    • Component/s: None
    • Labels:
      None

      Description

      We should rename YARN_HOME to HADOOP_YARN_HOME to be consistent with rest of Hadoop sub-projects.

      1. YARN-9-20120923.txt
        23 kB
        Vinod Kumar Vavilapalli
      2. YARN-9-20120912.txt
        17 kB
        Vinod Kumar Vavilapalli

        Issue Links

          Activity

          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk #1208 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1208/)
          YARN-9. Rename YARN_HOME to HADOOP_YARN_HOME. Contributed by Vinod K V. (Revision 1390218)

          Result = SUCCESS
          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1390218
          Files :

          • /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/slaves.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-config.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemons.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/conf/yarn-env.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/ApplicationConstants.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-default.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LinuxContainerExecutor.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/CapacityScheduler.apt.vm
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/ClusterSetup.apt.vm
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/SingleCluster.apt.vm
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #1208 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1208/ ) YARN-9 . Rename YARN_HOME to HADOOP_YARN_HOME. Contributed by Vinod K V. (Revision 1390218) Result = SUCCESS acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1390218 Files : /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/slaves.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-config.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemons.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/conf/yarn-env.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/ApplicationConstants.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-default.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LinuxContainerExecutor.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/CapacityScheduler.apt.vm /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/ClusterSetup.apt.vm /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/SingleCluster.apt.vm
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk #1177 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1177/)
          YARN-9. Rename YARN_HOME to HADOOP_YARN_HOME. Contributed by Vinod K V. (Revision 1390218)

          Result = SUCCESS
          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1390218
          Files :

          • /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/slaves.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-config.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemons.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/conf/yarn-env.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/ApplicationConstants.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-default.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LinuxContainerExecutor.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/CapacityScheduler.apt.vm
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/ClusterSetup.apt.vm
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/SingleCluster.apt.vm
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk #1177 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1177/ ) YARN-9 . Rename YARN_HOME to HADOOP_YARN_HOME. Contributed by Vinod K V. (Revision 1390218) Result = SUCCESS acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1390218 Files : /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/slaves.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-config.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemons.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/conf/yarn-env.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/ApplicationConstants.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-default.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LinuxContainerExecutor.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/CapacityScheduler.apt.vm /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/ClusterSetup.apt.vm /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/SingleCluster.apt.vm
          Hide
          Vinod Kumar Vavilapalli added a comment -

          Tx for the review and verification, Arun!

          Show
          Vinod Kumar Vavilapalli added a comment - Tx for the review and verification, Arun!
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk-Commit #2773 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/2773/)
          YARN-9. Rename YARN_HOME to HADOOP_YARN_HOME. Contributed by Vinod K V. (Revision 1390218)

          Result = SUCCESS
          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1390218
          Files :

          • /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/slaves.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-config.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemons.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/conf/yarn-env.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/ApplicationConstants.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-default.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LinuxContainerExecutor.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/CapacityScheduler.apt.vm
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/ClusterSetup.apt.vm
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/SingleCluster.apt.vm
          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #2773 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/2773/ ) YARN-9 . Rename YARN_HOME to HADOOP_YARN_HOME. Contributed by Vinod K V. (Revision 1390218) Result = SUCCESS acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1390218 Files : /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/slaves.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-config.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemons.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/conf/yarn-env.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/ApplicationConstants.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-default.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LinuxContainerExecutor.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/CapacityScheduler.apt.vm /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/ClusterSetup.apt.vm /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/SingleCluster.apt.vm
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk-Commit #2836 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/2836/)
          YARN-9. Rename YARN_HOME to HADOOP_YARN_HOME. Contributed by Vinod K V. (Revision 1390218)

          Result = SUCCESS
          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1390218
          Files :

          • /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/slaves.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-config.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemons.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/conf/yarn-env.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/ApplicationConstants.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-default.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LinuxContainerExecutor.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/CapacityScheduler.apt.vm
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/ClusterSetup.apt.vm
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/SingleCluster.apt.vm
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk-Commit #2836 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/2836/ ) YARN-9 . Rename YARN_HOME to HADOOP_YARN_HOME. Contributed by Vinod K V. (Revision 1390218) Result = SUCCESS acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1390218 Files : /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/slaves.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-config.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemons.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/conf/yarn-env.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/ApplicationConstants.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-default.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LinuxContainerExecutor.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/CapacityScheduler.apt.vm /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/ClusterSetup.apt.vm /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/SingleCluster.apt.vm
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk-Commit #2795 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/2795/)
          YARN-9. Rename YARN_HOME to HADOOP_YARN_HOME. Contributed by Vinod K V. (Revision 1390218)

          Result = FAILURE
          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1390218
          Files :

          • /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/slaves.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-config.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemons.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/conf/yarn-env.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/ApplicationConstants.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-default.xml
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LinuxContainerExecutor.java
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/CapacityScheduler.apt.vm
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/ClusterSetup.apt.vm
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/SingleCluster.apt.vm
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk-Commit #2795 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/2795/ ) YARN-9 . Rename YARN_HOME to HADOOP_YARN_HOME. Contributed by Vinod K V. (Revision 1390218) Result = FAILURE acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1390218 Files : /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/slaves.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-config.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemons.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/conf/yarn-env.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/ApplicationConstants.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-default.xml /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LinuxContainerExecutor.java /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/CapacityScheduler.apt.vm /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/ClusterSetup.apt.vm /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/apt/SingleCluster.apt.vm
          Hide
          Arun C Murthy added a comment -

          I just committed this. Thanks Vinod!

          Show
          Arun C Murthy added a comment - I just committed this. Thanks Vinod!
          Hide
          Arun C Murthy added a comment -

          +1 tested on both single-node (non-secure) and 5 node secure clusters. I'm committing this.

          Show
          Arun C Murthy added a comment - +1 tested on both single-node (non-secure) and 5 node secure clusters. I'm committing this.
          Hide
          Arun C Murthy added a comment -

          The three patches look fine, I'm testing them on a cluster and I'll commit after.

          Show
          Arun C Murthy added a comment - The three patches look fine, I'm testing them on a cluster and I'll commit after.
          Hide
          Hemanth Yamijala added a comment -

          Vinod, verified the changes in YARN and Common projects. Also checked if there are any leftover occurrences of YARN_HOME. Both of these look fine to me. So +1

          The changes in MAPREDUCE-4649 look more involved. Will probably take more time or a more knowledgeable person to review that.

          Show
          Hemanth Yamijala added a comment - Vinod, verified the changes in YARN and Common projects. Also checked if there are any leftover occurrences of YARN_HOME. Both of these look fine to me. So +1 The changes in MAPREDUCE-4649 look more involved. Will probably take more time or a more knowledgeable person to review that.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12546224/YARN-9-20120923.txt
          against trunk revision .

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

          -1 tests included. 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. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. The javadoc tool did not generate any 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-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site.

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

          Test results: https://builds.apache.org/job/PreCommit-YARN-Build/47//testReport/
          Console output: https://builds.apache.org/job/PreCommit-YARN-Build/47//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/12546224/YARN-9-20120923.txt against trunk revision . +1 @author. The patch does not contain any @author tags. -1 tests included. 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. The applied patch does not increase the total number of javac compiler warnings. +1 javadoc. The javadoc tool did not generate any 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-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site. +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-YARN-Build/47//testReport/ Console output: https://builds.apache.org/job/PreCommit-YARN-Build/47//console This message is automatically generated.
          Hide
          Vinod Kumar Vavilapalli added a comment -

          Updated patch, ready for review.

          Hemanth and Tucu, appreciate feedback on this latest patch.

          FYI, I've tested this on my local setup together with the latest patches at HADOOP-8794 and MAPREDUCE-4649.

          Show
          Vinod Kumar Vavilapalli added a comment - Updated patch, ready for review. Hemanth and Tucu, appreciate feedback on this latest patch. FYI, I've tested this on my local setup together with the latest patches at HADOOP-8794 and MAPREDUCE-4649 .
          Hide
          Vinod Kumar Vavilapalli added a comment -

          Also there are other YARN_ env vars, they should all be renamed to HADOOP_YARN_.

          Tucu, I'd would like to do this separately in an immediate follow-up.

          Show
          Vinod Kumar Vavilapalli added a comment - Also there are other YARN_ env vars, they should all be renamed to HADOOP_YARN_. Tucu, I'd would like to do this separately in an immediate follow-up.
          Hide
          Vinod Kumar Vavilapalli added a comment -

          So does that mean that we need to merge this into branch-0.23? Because I plan on beating on 0.23 until it is "deemed stable" and it does not have this non-backwards compatible change in it.

          Yes Bobby, that would be better. I'd favor that.

          But at some point we need to start getting very serious about maintaining backwards compatibility even on the small things.

          Agreed. All the more reasons, we should fix incompatible changes which we need as early as possible. Arguably, this change can be made compatible, but I believe will only add cruft given we haven't gone beyond alpha-level stability.

          Show
          Vinod Kumar Vavilapalli added a comment - So does that mean that we need to merge this into branch-0.23? Because I plan on beating on 0.23 until it is "deemed stable" and it does not have this non-backwards compatible change in it. Yes Bobby, that would be better. I'd favor that. But at some point we need to start getting very serious about maintaining backwards compatibility even on the small things. Agreed. All the more reasons, we should fix incompatible changes which we need as early as possible. Arguably, this change can be made compatible, but I believe will only add cruft given we haven't gone beyond alpha-level stability.
          Hide
          Robert Joseph Evans added a comment -

          Yes, you are right. But I don't think we need to support YARN_HOME since, like Hitesh mentioned, none of the releases so far are deemed stable.

          So does that mean that we need to merge this into branch-0.23? Because I plan on beating on 0.23 until it is "deemed stable" and it does not have this non-backwards compatible change in it. I get that you want to clean things up and I am OK with that. But at some point we need to start getting very serious about maintaining backwards compatibility even on the small things.

          Show
          Robert Joseph Evans added a comment - Yes, you are right. But I don't think we need to support YARN_HOME since, like Hitesh mentioned, none of the releases so far are deemed stable. So does that mean that we need to merge this into branch-0.23? Because I plan on beating on 0.23 until it is "deemed stable" and it does not have this non-backwards compatible change in it. I get that you want to clean things up and I am OK with that. But at some point we need to start getting very serious about maintaining backwards compatibility even on the small things.
          Hide
          Vinod Kumar Vavilapalli added a comment -

          I'm getting CNF exceptions, I'll check a few other settings in my ENV to verify exactly why.

          Please check if MR jobs are working fine. Also do apply the patch at HADOOP-8794, it perhaps misses a few more changes but has the bare minimum changes needed already.

          Show
          Vinod Kumar Vavilapalli added a comment - I'm getting CNF exceptions, I'll check a few other settings in my ENV to verify exactly why. Please check if MR jobs are working fine. Also do apply the patch at HADOOP-8794 , it perhaps misses a few more changes but has the bare minimum changes needed already.
          Hide
          Alejandro Abdelnur added a comment -

          I'm getting CNF exceptions, I'll check a few other settings in my ENV to verify exactly why.

          Show
          Alejandro Abdelnur added a comment - I'm getting CNF exceptions, I'll check a few other settings in my ENV to verify exactly why.
          Hide
          Vinod Kumar Vavilapalli added a comment -

          Tx for the reviews, Hemanth and Alejandro.

          I split the common changes - see HADOOP-8794. Will update that patch with any missing updates.

          These three seem to be just documentation updates:

          Will fix the doc updates.

          Will fix the other stuff too, this was the bare minimum I needed to make the jobs work.

          Also there are other YARN_ env vars, they should all be renamed to HADOOP_YARN_.

          I wanted to do that separately, will revisit this depending on the final size of this patch.

          I'm getting failures with custom AMs, I believe it is due to the changes not being through out the code/scripts/config files.

          Distributed-shell and MR jobs are working for me. Any peculiarities in your custom AMs?

          Show
          Vinod Kumar Vavilapalli added a comment - Tx for the reviews, Hemanth and Alejandro. I split the common changes - see HADOOP-8794 . Will update that patch with any missing updates. These three seem to be just documentation updates: Will fix the doc updates. Will fix the other stuff too, this was the bare minimum I needed to make the jobs work. Also there are other YARN_ env vars, they should all be renamed to HADOOP_YARN_. I wanted to do that separately, will revisit this depending on the final size of this patch. I'm getting failures with custom AMs, I believe it is due to the changes not being through out the code/scripts/config files. Distributed-shell and MR jobs are working for me. Any peculiarities in your custom AMs?
          Hide
          Alejandro Abdelnur added a comment - - edited

          My bad on the +1. There are several place where it has not been updated. Also there are other YARN_ env vars, they should all be renamed to HADOOP_YARN_. Plus, with the patch, I'm getting failures with custom AMs, I believe it is due to the changes not being through out the code/scripts/config files.

          Show
          Alejandro Abdelnur added a comment - - edited My bad on the +1. There are several place where it has not been updated. Also there are other YARN_ env vars, they should all be renamed to HADOOP_YARN_. Plus, with the patch, I'm getting failures with custom AMs, I believe it is due to the changes not being through out the code/scripts/config files.
          Hide
          Hemanth Yamijala added a comment -

          Hi Vinod,

          Couple of places still seem to have YARN_HOME. Should these be changed too ?

          ./hadoop-common-project/hadoop-common/src/main/bin/hadoop-config.sh
          ./hadoop-common-project/hadoop-common/src/main/bin/start-all.sh
          ./hadoop-mapreduce-project/bin/mr-jobhistory-daemon.sh (This, I think, is being addressed by MAPREDUCE-4649)
          ./hadoop-mapreduce-project/INSTALL (looks like documentation, and possibly needs to be changed to HADOOP_MAPRED_HOME ?)

          These three seem to be just documentation updates:
          ./hadoop-yarn-project/hadoop-yarn/bin/slaves.sh
          ./hadoop-yarn-project/hadoop-yarn/bin/yarn
          ./hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh

          ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/ApplicationConstants.java (I think the Enum name can be changed to be consistent with the actual value)
          ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java
          ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-default.xml (default exported values)
          ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LinuxContainerExecutor.java

          Show
          Hemanth Yamijala added a comment - Hi Vinod, Couple of places still seem to have YARN_HOME. Should these be changed too ? ./hadoop-common-project/hadoop-common/src/main/bin/hadoop-config.sh ./hadoop-common-project/hadoop-common/src/main/bin/start-all.sh ./hadoop-mapreduce-project/bin/mr-jobhistory-daemon.sh (This, I think, is being addressed by MAPREDUCE-4649 ) ./hadoop-mapreduce-project/INSTALL (looks like documentation, and possibly needs to be changed to HADOOP_MAPRED_HOME ?) These three seem to be just documentation updates: ./hadoop-yarn-project/hadoop-yarn/bin/slaves.sh ./hadoop-yarn-project/hadoop-yarn/bin/yarn ./hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/ApplicationConstants.java (I think the Enum name can be changed to be consistent with the actual value) ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/resources/yarn-default.xml (default exported values) ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LinuxContainerExecutor.java
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12544867/YARN-9-20120912.txt
          against trunk revision .

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

          -1 tests included. 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. The applied patch does not increase the total number of javac compiler warnings.

          +1 javadoc. The javadoc tool did not generate any 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-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site.

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

          Test results: https://builds.apache.org/job/PreCommit-YARN-Build/36//testReport/
          Console output: https://builds.apache.org/job/PreCommit-YARN-Build/36//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/12544867/YARN-9-20120912.txt against trunk revision . +1 @author. The patch does not contain any @author tags. -1 tests included. 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. The applied patch does not increase the total number of javac compiler warnings. +1 javadoc. The javadoc tool did not generate any 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-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site. +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-YARN-Build/36//testReport/ Console output: https://builds.apache.org/job/PreCommit-YARN-Build/36//console This message is automatically generated.
          Hide
          Alejandro Abdelnur added a comment -

          +1, makes sense. yes, opening jiras in Bigtop would help to signal them of the coming changes. thx.

          Show
          Alejandro Abdelnur added a comment - +1, makes sense. yes, opening jiras in Bigtop would help to signal them of the coming changes. thx.
          Hide
          Vinod Kumar Vavilapalli added a comment -

          FYI: Bigtop will have to adapt to this change, but I'm +/-0 on whether YARN_HOME needs to be supported. I suppose some existing users might have it hardcoded.

          Yes, you are right. But I don't think we need to support YARN_HOME since, like Hitesh mentioned, none of the releases so far are deemed stable.

          I understand some users will need update their scripts, but it's good we are doing it early in the releases.

          FYI, there are few more like issues like https://issues.apache.org/jira/browse/MAPREDUCE-4649 which will impact Bigtop. How do you think we should drive this? I can just open tickets on Bigtop JIRA if that is fine.

          Show
          Vinod Kumar Vavilapalli added a comment - FYI: Bigtop will have to adapt to this change, but I'm +/-0 on whether YARN_HOME needs to be supported. I suppose some existing users might have it hardcoded. Yes, you are right. But I don't think we need to support YARN_HOME since, like Hitesh mentioned, none of the releases so far are deemed stable. I understand some users will need update their scripts, but it's good we are doing it early in the releases. FYI, there are few more like issues like https://issues.apache.org/jira/browse/MAPREDUCE-4649 which will impact Bigtop. How do you think we should drive this? I can just open tickets on Bigtop JIRA if that is fine.
          Hide
          Hitesh Shah added a comment -

          @Alejandro, given that we are designating hadoop-2.0.x as alpha i.e. APIs may change/break, wouldn't it be better from a long-term simplicity point of view to just remove YARN_HOME?

          Show
          Hitesh Shah added a comment - @Alejandro, given that we are designating hadoop-2.0.x as alpha i.e. APIs may change/break, wouldn't it be better from a long-term simplicity point of view to just remove YARN_HOME?
          Hide
          Roman Shaposhnik added a comment -

          FYI: Bigtop will have to adapt to this change, but I'm +/-0 on whether YARN_HOME needs to be supported. I suppose some existing users might have it hardcoded.

          Show
          Roman Shaposhnik added a comment - FYI: Bigtop will have to adapt to this change, but I'm +/-0 on whether YARN_HOME needs to be supported. I suppose some existing users might have it hardcoded.
          Hide
          Alejandro Abdelnur added a comment -

          shouldn't we still handle YARN_HOME as deprecated?

          Show
          Alejandro Abdelnur added a comment - shouldn't we still handle YARN_HOME as deprecated?
          Hide
          Vinod Kumar Vavilapalli added a comment -

          Simple find and replace.

          Tested this by bringing up a single node YARN cluster and verified by running the dist-shell app successfully.

          Show
          Vinod Kumar Vavilapalli added a comment - Simple find and replace. Tested this by bringing up a single node YARN cluster and verified by running the dist-shell app successfully.
          Hide
          Vinod Kumar Vavilapalli added a comment -

          I am fixing this, better sooner than later.

          Show
          Vinod Kumar Vavilapalli added a comment - I am fixing this, better sooner than later.

            People

            • Assignee:
              Vinod Kumar Vavilapalli
              Reporter:
              Arun C Murthy
            • Votes:
              0 Vote for this issue
              Watchers:
              12 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development