Hadoop Common
  1. Hadoop Common
  2. HADOOP-9379

capture the ulimit info after printing the log to the console

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.2.0, 2.0.4-alpha
    • Fix Version/s: 1.2.0, 0.23.7, 2.1.0-beta, 2.0.4-alpha
    • Component/s: None
    • Labels:
      None

      Description

      Based on the discussions in HADOOP-9253 people prefer if we dont print the ulimit info to the console but still have it in the logs.

      Just need to move the head statement to before the capture of ulimit code.

      1. HADOOP-9379.patch
        1 kB
        Arpit Gupta
      2. HADOOP-9379.branch-1.patch
        0.7 kB
        Arpit Gupta

        Issue Links

          Activity

          Hide
          Arpit Gupta added a comment -

          Patch for trunk

          Show
          Arpit Gupta added a comment - Patch for trunk
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12572552/HADOOP-9379.patch
          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-common-project/hadoop-common.

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

          Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2289//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2289//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/12572552/HADOOP-9379.patch 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-common-project/hadoop-common. +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2289//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2289//console This message is automatically generated.
          Hide
          Arpit Gupta added a comment -

          No tests added as this is a change to the shell scripts. Manually verified that the ulimit info is only in the logs.

          Show
          Arpit Gupta added a comment - No tests added as this is a change to the shell scripts. Manually verified that the ulimit info is only in the logs.
          Hide
          Junping Du added a comment -

          Verified it works well on my environment. Thanks Arpit.

          Show
          Junping Du added a comment - Verified it works well on my environment. Thanks Arpit.
          Hide
          Suresh Srinivas added a comment -

          I committed the patch to trunk, branch-2, branch-1 and branch-1.2.

          Thank you Arpit! Thanks to Junping for doing quick validation.

          Show
          Suresh Srinivas added a comment - I committed the patch to trunk, branch-2, branch-1 and branch-1.2. Thank you Arpit! Thanks to Junping for doing quick validation.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-trunk-Commit #3447 (See https://builds.apache.org/job/Hadoop-trunk-Commit/3447/)
          HADOOP-9379. capture the ulimit info after printing the log to the console. Contributed by Arpit Gupta. (Revision 1455316)

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

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/bin/hadoop-daemon.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          Show
          Hudson added a comment - Integrated in Hadoop-trunk-Commit #3447 (See https://builds.apache.org/job/Hadoop-trunk-Commit/3447/ ) HADOOP-9379 . capture the ulimit info after printing the log to the console. Contributed by Arpit Gupta. (Revision 1455316) Result = SUCCESS suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1455316 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/bin/hadoop-daemon.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Yarn-trunk #153 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/153/)
          HADOOP-9379. capture the ulimit info after printing the log to the console. Contributed by Arpit Gupta. (Revision 1455316)

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

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/bin/hadoop-daemon.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          Show
          Hudson added a comment - Integrated in Hadoop-Yarn-trunk #153 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/153/ ) HADOOP-9379 . capture the ulimit info after printing the log to the console. Contributed by Arpit Gupta. (Revision 1455316) Result = SUCCESS suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1455316 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/bin/hadoop-daemon.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk #1342 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1342/)
          HADOOP-9379. capture the ulimit info after printing the log to the console. Contributed by Arpit Gupta. (Revision 1455316)

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

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/bin/hadoop-daemon.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk #1342 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1342/ ) HADOOP-9379 . capture the ulimit info after printing the log to the console. Contributed by Arpit Gupta. (Revision 1455316) Result = FAILURE suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1455316 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/bin/hadoop-daemon.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          Hide
          Thomas Graves added a comment -

          merged into branch-0.23

          Show
          Thomas Graves added a comment - merged into branch-0.23
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk #1370 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1370/)
          HADOOP-9379. capture the ulimit info after printing the log to the console. Contributed by Arpit Gupta. (Revision 1455316)

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

          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/bin/hadoop-daemon.sh
          • /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #1370 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1370/ ) HADOOP-9379 . capture the ulimit info after printing the log to the console. Contributed by Arpit Gupta. (Revision 1455316) Result = SUCCESS suresh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1455316 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/bin/hadoop-daemon.sh /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-0.23-Build #552 (See https://builds.apache.org/job/Hadoop-Hdfs-0.23-Build/552/)
          HADOOP-9379. capture the ulimit info after printing the log to the console. (Arpit Gupta via suresh) (Revision 1455519)

          Result = UNSTABLE
          tgraves : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1455519
          Files :

          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/bin/hadoop-daemon.sh
          • /hadoop/common/branches/branch-0.23/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-0.23-Build #552 (See https://builds.apache.org/job/Hadoop-Hdfs-0.23-Build/552/ ) HADOOP-9379 . capture the ulimit info after printing the log to the console. (Arpit Gupta via suresh) (Revision 1455519) Result = UNSTABLE tgraves : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1455519 Files : /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/bin/hadoop-daemon.sh /hadoop/common/branches/branch-0.23/hadoop-yarn-project/hadoop-yarn/bin/yarn-daemon.sh
          Hide
          Konstantin Boudnik added a comment -

          It is marked for 2.0.4-alpha, but never made it in. Committing.

          Show
          Konstantin Boudnik added a comment - It is marked for 2.0.4-alpha, but never made it in. Committing.

            People

            • Assignee:
              Arpit Gupta
              Reporter:
              Arpit Gupta
            • Votes:
              0 Vote for this issue
              Watchers:
              9 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development