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

Documentation refers to incorrect nodemanager health checker interval property

        Details

        • Target Version/s:
        • Hadoop Flags:
          Reviewed

          Description

          I think one parameter in the "Monitoring Health of NodeManagers" section of "Cluster Setup" is wrong.The parameter "yarn.nodemanager.health-checker.script.interval-ms" should be “yarn.nodemanager.health-checker.interval-ms”.See http://hadoop.apache.org/docs/r2.7.3/hadoop-project-dist/hadoop-common/ClusterSetup.html.

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            cheersyang Weiwei Yang added a comment -

            Thanks Charles Zhang to report this issue, I just uploaded a simple patch to fix this typo.

            Show
            cheersyang Weiwei Yang added a comment - Thanks Charles Zhang to report this issue, I just uploaded a simple patch to fix this typo.
            Hide
            Permalink
            rebeyond1218 Charles Zhang added a comment -

            Thinks, Weiwei Yang !

            Show
            rebeyond1218 Charles Zhang added a comment - Thinks, Weiwei Yang !
            Hide
            Permalink
            hadoopqa Hadoop QA added a comment -
            +1 overall



            Vote Subsystem Runtime Comment
            0 reexec 0m 13s Docker mode activated.
            +1 @author 0m 0s The patch does not contain any @author tags.
            +1 mvninstall 12m 28s trunk passed
            +1 mvnsite 0m 55s trunk passed
            +1 mvnsite 0m 51s the patch passed
            +1 whitespace 0m 0s The patch has no whitespace issues.
            +1 asflicense 0m 16s The patch does not generate ASF License warnings.
            15m 22s



            Subsystem Report/Notes
            Docker Image:yetus/hadoop:a9ad5d6
            JIRA Issue YARN-6274
            JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12855801/YARN-6274.01.patch
            Optional Tests asflicense mvnsite
            uname Linux 62847fb920d9 3.13.0-106-generic #153-Ubuntu SMP Tue Dec 6 15:44:32 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
            Build tool maven
            Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
            git revision trunk / 19a1fc6
            modules C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common
            Console output https://builds.apache.org/job/PreCommit-YARN-Build/15149/console
            Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org

            This message was automatically generated.

            Show
            hadoopqa Hadoop QA added a comment - +1 overall Vote Subsystem Runtime Comment 0 reexec 0m 13s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 mvninstall 12m 28s trunk passed +1 mvnsite 0m 55s trunk passed +1 mvnsite 0m 51s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 asflicense 0m 16s The patch does not generate ASF License warnings. 15m 22s Subsystem Report/Notes Docker Image:yetus/hadoop:a9ad5d6 JIRA Issue YARN-6274 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12855801/YARN-6274.01.patch Optional Tests asflicense mvnsite uname Linux 62847fb920d9 3.13.0-106-generic #153-Ubuntu SMP Tue Dec 6 15:44:32 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 19a1fc6 modules C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common Console output https://builds.apache.org/job/PreCommit-YARN-Build/15149/console Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
            Hide
            Permalink
            jlowe Jason Lowe added a comment -

            Thanks for the report, Charles Zhang and for the patch, Weiwei Yang!

            Note that in the future the Fix Version should only be set by the committer when the patch is committed, as that field indicates what versions actually have the fix. The Target Version field should be used to indicate which versions are being targeted to fix.

            +1 patch lgtm. Committing this.

            Show
            jlowe Jason Lowe added a comment - Thanks for the report, Charles Zhang and for the patch, Weiwei Yang ! Note that in the future the Fix Version should only be set by the committer when the patch is committed, as that field indicates what versions actually have the fix. The Target Version field should be used to indicate which versions are being targeted to fix. +1 patch lgtm. Committing this.
            Hide
            Permalink
            jlowe Jason Lowe added a comment -

            Thanks, Weiwei Yang! I committed this to trunk, branch-2, branch-2.8, branch-2.8.0, and branch-2.7.

            Show
            jlowe Jason Lowe added a comment - Thanks, Weiwei Yang ! I committed this to trunk, branch-2, branch-2.8, branch-2.8.0, and branch-2.7.
            Hide
            Permalink
            cheersyang Weiwei Yang added a comment -

            Thanks Jason Lowe for the quick review and commit!

            Show
            cheersyang Weiwei Yang added a comment - Thanks Jason Lowe for the quick review and commit!
            Hide
            Permalink
            hudson Hudson added a comment -

            FAILURE: Integrated in Jenkins build Hadoop-trunk-Commit #11339 (See https://builds.apache.org/job/Hadoop-trunk-Commit/11339/)
            YARN-6274. Documentation refers to incorrect nodemanager health checker (jlowe: rev 05237636d3a359dc502e8fa2d66bfdcea274f86c)

            • (edit) hadoop-common-project/hadoop-common/src/site/markdown/ClusterSetup.md
            Show
            hudson Hudson added a comment - FAILURE: Integrated in Jenkins build Hadoop-trunk-Commit #11339 (See https://builds.apache.org/job/Hadoop-trunk-Commit/11339/ ) YARN-6274 . Documentation refers to incorrect nodemanager health checker (jlowe: rev 05237636d3a359dc502e8fa2d66bfdcea274f86c) (edit) hadoop-common-project/hadoop-common/src/site/markdown/ClusterSetup.md

              People

              • Assignee:
                cheersyang Weiwei Yang
                Reporter:
                rebeyond1218 Charles Zhang
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development