Details

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

      Description

      src/test/system/aop/org/apache/hadoop/mapred/TaskAspect.aj:72 [warning] advice defined in org.apache.hadoop.mapred.TaskAspect has not been applied [Xlint:adviceDidNotMatch]

      After the fix in MR-2429, the call to ping in TaskAspect needs to be fixed.

      1. MR2443.patch
        2 kB
        Siddharth Seth

        Issue Links

          Activity

          Hide
          Siddharth Seth added a comment -

          Patch for FI fix. Also includes a comment to TaskUmbilicalProtocol.java

          Show
          Siddharth Seth added a comment - Patch for FI fix. Also includes a comment to TaskUmbilicalProtocol.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/12476678/MR2443.patch
          against trunk revision 1094093.

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

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

          -1 patch. The patch command could not apply the patch.

          Console output: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/174//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/12476678/MR2443.patch against trunk revision 1094093. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 3 new or modified tests. -1 patch. The patch command could not apply the patch. Console output: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/174//console This message is automatically generated.
          Hide
          Tsz Wo Nicholas Sze added a comment -

          +1 patch looks good.

          I have verified that warning is gone after the patch.

          Show
          Tsz Wo Nicholas Sze added a comment - +1 patch looks good. I have verified that warning is gone after the patch.
          Hide
          Tsz Wo Nicholas Sze added a comment -

          I have committed this. Thanks, Siddharth!

          Show
          Tsz Wo Nicholas Sze added a comment - I have committed this. Thanks, Siddharth!
          Hide
          Owen O'Malley added a comment -

          Hadoop 0.20.204.0 was just released.

          Show
          Owen O'Malley added a comment - Hadoop 0.20.204.0 was just released.

            People

            • Assignee:
              Siddharth Seth
              Reporter:
              Siddharth Seth
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development