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

FSAppAttempt demand calculation considers demands at multiple locality levels different

        Details

        • Type: Bug
        • Status: Resolved
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: 2.8.0, 2.7.2
        • Fix Version/s: 2.9.0, 3.0.0-alpha1
        • Component/s: fairscheduler
        • Labels:
          None
        • Hadoop Flags:
          Reviewed

          Description

          FSAppAttempt adds duplicate ResourceRequest to demand in updateDemand. We should only count ResourceRequest for ResourceRequest.ANY when calculate demand.
          Because hasContainerForNode will return false if no container request for ResourceRequest.ANY and both allocateNodeLocal and allocateRackLocal will also decrease the number of containers for ResourceRequest.ANY.
          This issue may cause current memory demand overflow(integer) because duplicate requests can be on multiple nodes.

            Issue Links

            is related to

            Bug - A problem which impairs or prevents the functions of the product. YARN-1458 FairScheduler: Zero weight can lead to livelock

            • Major - Major loss of function.
            • Resolved

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              hadoopqa Hadoop QA added a comment -
              -1 overall



              Vote Subsystem Runtime Comment
              0 reexec 0m 11s Docker mode activated.
              +1 @author 0m 0s The patch does not contain any @author tags.
              +1 test4tests 0m 0s The patch appears to include 1 new or modified test files.
              +1 mvninstall 6m 31s trunk passed
              +1 compile 0m 25s trunk passed with JDK v1.8.0_77
              +1 compile 0m 29s trunk passed with JDK v1.7.0_95
              +1 checkstyle 0m 17s trunk passed
              +1 mvnsite 0m 34s trunk passed
              +1 mvneclipse 0m 14s trunk passed
              +1 findbugs 1m 4s trunk passed
              +1 javadoc 0m 21s trunk passed with JDK v1.8.0_77
              +1 javadoc 0m 26s trunk passed with JDK v1.7.0_95
              +1 mvninstall 0m 30s the patch passed
              +1 compile 0m 26s the patch passed with JDK v1.8.0_77
              +1 javac 0m 26s the patch passed
              +1 compile 0m 27s the patch passed with JDK v1.7.0_95
              +1 javac 0m 27s the patch passed
              +1 checkstyle 0m 16s the patch passed
              +1 mvnsite 0m 32s the patch passed
              +1 mvneclipse 0m 14s the patch passed
              +1 whitespace 0m 0s Patch has no whitespace issues.
              +1 findbugs 1m 14s the patch passed
              +1 javadoc 0m 18s the patch passed with JDK v1.8.0_77
              +1 javadoc 0m 24s the patch passed with JDK v1.7.0_95
              -1 unit 75m 43s hadoop-yarn-server-resourcemanager in the patch failed with JDK v1.8.0_77.
              -1 unit 75m 56s hadoop-yarn-server-resourcemanager in the patch failed with JDK v1.7.0_95.
              +1 asflicense 0m 16s Patch does not generate ASF License warnings.
              167m 46s



              Reason Tests
              JDK v1.8.0_77 Failed junit tests hadoop.yarn.server.resourcemanager.TestClientRMTokens
                hadoop.yarn.server.resourcemanager.TestAMAuthorization
              JDK v1.8.0_77 Timed out junit tests org.apache.hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesNodes
              JDK v1.7.0_95 Failed junit tests hadoop.yarn.server.resourcemanager.TestClientRMTokens
                hadoop.yarn.server.resourcemanager.TestAMAuthorization
              JDK v1.7.0_95 Timed out junit tests org.apache.hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesNodes



              Subsystem Report/Notes
              Docker Image:yetus/hadoop:fbe3e86
              JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12799911/YARN-4979.001.patch
              JIRA Issue YARN-4979
              Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
              uname Linux 245d1092cef5 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
              Build tool maven
              Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
              git revision trunk / 5bd7b59
              Default Java 1.7.0_95
              Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_77 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_95
              findbugs v3.0.0
              unit https://builds.apache.org/job/PreCommit-YARN-Build/11155/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdk1.8.0_77.txt
              unit https://builds.apache.org/job/PreCommit-YARN-Build/11155/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdk1.7.0_95.txt
              unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/11155/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdk1.8.0_77.txt https://builds.apache.org/job/PreCommit-YARN-Build/11155/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdk1.7.0_95.txt
              JDK v1.7.0_95 Test Results https://builds.apache.org/job/PreCommit-YARN-Build/11155/testReport/
              modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager
              Console output https://builds.apache.org/job/PreCommit-YARN-Build/11155/console
              Powered by Apache Yetus 0.2.0 http://yetus.apache.org

              This message was automatically generated.

              Show
              hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 11s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 1 new or modified test files. +1 mvninstall 6m 31s trunk passed +1 compile 0m 25s trunk passed with JDK v1.8.0_77 +1 compile 0m 29s trunk passed with JDK v1.7.0_95 +1 checkstyle 0m 17s trunk passed +1 mvnsite 0m 34s trunk passed +1 mvneclipse 0m 14s trunk passed +1 findbugs 1m 4s trunk passed +1 javadoc 0m 21s trunk passed with JDK v1.8.0_77 +1 javadoc 0m 26s trunk passed with JDK v1.7.0_95 +1 mvninstall 0m 30s the patch passed +1 compile 0m 26s the patch passed with JDK v1.8.0_77 +1 javac 0m 26s the patch passed +1 compile 0m 27s the patch passed with JDK v1.7.0_95 +1 javac 0m 27s the patch passed +1 checkstyle 0m 16s the patch passed +1 mvnsite 0m 32s the patch passed +1 mvneclipse 0m 14s the patch passed +1 whitespace 0m 0s Patch has no whitespace issues. +1 findbugs 1m 14s the patch passed +1 javadoc 0m 18s the patch passed with JDK v1.8.0_77 +1 javadoc 0m 24s the patch passed with JDK v1.7.0_95 -1 unit 75m 43s hadoop-yarn-server-resourcemanager in the patch failed with JDK v1.8.0_77. -1 unit 75m 56s hadoop-yarn-server-resourcemanager in the patch failed with JDK v1.7.0_95. +1 asflicense 0m 16s Patch does not generate ASF License warnings. 167m 46s Reason Tests JDK v1.8.0_77 Failed junit tests hadoop.yarn.server.resourcemanager.TestClientRMTokens   hadoop.yarn.server.resourcemanager.TestAMAuthorization JDK v1.8.0_77 Timed out junit tests org.apache.hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesNodes JDK v1.7.0_95 Failed junit tests hadoop.yarn.server.resourcemanager.TestClientRMTokens   hadoop.yarn.server.resourcemanager.TestAMAuthorization JDK v1.7.0_95 Timed out junit tests org.apache.hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesNodes Subsystem Report/Notes Docker Image:yetus/hadoop:fbe3e86 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12799911/YARN-4979.001.patch JIRA Issue YARN-4979 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 245d1092cef5 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 5bd7b59 Default Java 1.7.0_95 Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_77 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_95 findbugs v3.0.0 unit https://builds.apache.org/job/PreCommit-YARN-Build/11155/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdk1.8.0_77.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/11155/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdk1.7.0_95.txt unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/11155/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdk1.8.0_77.txt https://builds.apache.org/job/PreCommit-YARN-Build/11155/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdk1.7.0_95.txt JDK v1.7.0_95 Test Results https://builds.apache.org/job/PreCommit-YARN-Build/11155/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager Console output https://builds.apache.org/job/PreCommit-YARN-Build/11155/console Powered by Apache Yetus 0.2.0 http://yetus.apache.org This message was automatically generated.
              Hide
              Permalink
              kasha Karthik Kambatla added a comment -

              More importantly, the demand could be wrong and affect the way fairshare is set?

              Show
              kasha Karthik Kambatla added a comment - More importantly, the demand could be wrong and affect the way fairshare is set?
              Hide
              Permalink
              kasha Karthik Kambatla added a comment -

              +1, checking this in.

              Show
              kasha Karthik Kambatla added a comment - +1, checking this in.
              Hide
              Permalink
              kasha Karthik Kambatla added a comment -

              Thanks for reporting and fixing this, Zhihai. Just committed this to trunk and branch-2.

              Show
              kasha Karthik Kambatla added a comment - Thanks for reporting and fixing this, Zhihai. Just committed this to trunk and branch-2.
              Hide
              Permalink
              hudson Hudson added a comment -

              SUCCESS: Integrated in Hadoop-trunk-Commit #9844 (See https://builds.apache.org/job/Hadoop-trunk-Commit/9844/)
              YARN-4979. FSAppAttempt demand calculation considers demands at multiple (kasha: rev 6d043aa4cf14bc88b1a19991e3bb0752499312b8)

              • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/TestFairScheduler.java
              • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/FSAppAttempt.java
              Show
              hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-trunk-Commit #9844 (See https://builds.apache.org/job/Hadoop-trunk-Commit/9844/ ) YARN-4979 . FSAppAttempt demand calculation considers demands at multiple (kasha: rev 6d043aa4cf14bc88b1a19991e3bb0752499312b8) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/TestFairScheduler.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/FSAppAttempt.java
              Hide
              Permalink
              zxu zhihai xu added a comment -

              thanks Karthik Kambatla for reviewing and committing the patch!

              Show
              zxu zhihai xu added a comment - thanks Karthik Kambatla for reviewing and committing the patch!

                People

                • Assignee:
                  zxu zhihai xu
                  Reporter:
                  zxu zhihai xu
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  5 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development