Hadoop Common
  1. Hadoop Common
  2. HADOOP-6828

Herrior uses old way of accessing logs directories

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.21.0
    • Fix Version/s: 0.21.0
    • Component/s: test
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Between 0.20 and 0.21 the way of passing log directories location has been changed from using environment variables to passing system properties. Herrior code needs to address it as well.

      1. HADOOP-6828.patch
        0.9 kB
        Konstantin Boudnik

        Issue Links

          Activity

          Hide
          Konstantin Boudnik added a comment -

          Fixing the issue. Patch prepared by Sreekanth.

          Show
          Konstantin Boudnik added a comment - Fixing the issue. Patch prepared by Sreekanth.
          Hide
          Konstantin Boudnik added a comment -

          Linking this up to keep track of related issues.

          Show
          Konstantin Boudnik added a comment - Linking this up to keep track of related issues.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12447236/HADOOP-6828.patch
          against trunk revision 954647.

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

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

          -1 javadoc. The javadoc tool appears to have generated 1 warning messages.

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

          +1 findbugs. The patch does not introduce any new Findbugs warnings.

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

          -1 core tests. The patch failed core unit tests.

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

          Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/586/testReport/
          Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/586/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
          Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/586/artifact/trunk/build/test/checkstyle-errors.html
          Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/586/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/12447236/HADOOP-6828.patch against trunk revision 954647. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 3 new or modified tests. -1 javadoc. The javadoc tool appears to have generated 1 warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. -1 core tests. The patch failed core unit tests. +1 contrib tests. The patch passed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/586/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/586/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/586/artifact/trunk/build/test/checkstyle-errors.html Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/586/console This message is automatically generated.
          Hide
          Konstantin Boudnik added a comment -
          • Test failure is unrelated
            Problem with Trash.. Consider using -skipTrash option
            rmr: Cannot move "file:/grid/0/hudson/hudson-slave/workspace/Hadoop-Patch-h4.grid.sp2.yahoo.net/trunk/build/test/data/testTrash/user/test" to the trash, as it contains the trash
            
          • warnings seem to be coming either from SecurityUtil or KerberosName. I don't see any other warnings.
          Show
          Konstantin Boudnik added a comment - Test failure is unrelated Problem with Trash.. Consider using -skipTrash option rmr: Cannot move "file:/grid/0/hudson/hudson-slave/workspace/Hadoop-Patch-h4.grid.sp2.yahoo.net/trunk/build/test/data/testTrash/user/test" to the trash, as it contains the trash warnings seem to be coming either from SecurityUtil or KerberosName . I don't see any other warnings.
          Hide
          Konstantin Boudnik added a comment -

          +1 patch looks good.
          In the verification of MAPREDUCE-1774 I have seen that the patch is working as expected.
          I'm going to commit this by the end of the day unless someone has onjections.

          Show
          Konstantin Boudnik added a comment - +1 patch looks good. In the verification of MAPREDUCE-1774 I have seen that the patch is working as expected. I'm going to commit this by the end of the day unless someone has onjections.
          Hide
          Konstantin Boudnik added a comment -

          I have just committed this. Thanks Sreekanth.

          Show
          Konstantin Boudnik added a comment - I have just committed this. Thanks Sreekanth.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk-Commit #310 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk-Commit/310/)
          HADOOP-6828. Herrior uses old way of accessing logs directories. Contributed by Sreekanth Ramakrishnan

          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #310 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk-Commit/310/ ) HADOOP-6828 . Herrior uses old way of accessing logs directories. Contributed by Sreekanth Ramakrishnan
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk #369 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk/369/)
          HADOOP-6828. Herrior uses old way of accessing logs directories. Contributed by Sreekanth Ramakrishnan

          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk #369 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk/369/ ) HADOOP-6828 . Herrior uses old way of accessing logs directories. Contributed by Sreekanth Ramakrishnan

            People

            • Assignee:
              Sreekanth Ramakrishnan
              Reporter:
              Konstantin Boudnik
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development