Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.0.0
    • Fix Version/s: 1.1.0
    • Component/s: namenode
    • Labels:
      None
    • Target Version/s:
    • Hadoop Flags:
      Reviewed

      Description

      It would be useful to know whether an attempt to recover a block is failing because the block was already recovered (has a new GS) or the block is missing.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        18h 55m 1 Eli Collins 07/Dec/11 18:09
        Resolved Resolved Closed Closed
        315d 18m 1 Matt Foley 17/Oct/12 19:27
        Matt Foley made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Matt Foley added a comment -

        Closed upon release of Hadoop-1.1.0.

        Show
        Matt Foley added a comment - Closed upon release of Hadoop-1.1.0.
        Matt Foley made changes -
        Fix Version/s 1.1.0 [ 12317959 ]
        Hide
        Matt Foley added a comment -

        Eli, please update FixedVersion field to be consistent with TargetVersion, when resolving a bug. Thanks.

        Show
        Matt Foley added a comment - Eli, please update FixedVersion field to be consistent with TargetVersion, when resolving a bug. Thanks.
        Eli Collins made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Hadoop Flags Reviewed [ 10343 ]
        Resolution Fixed [ 1 ]
        Hide
        Eli Collins added a comment -

        Thanks Todd, I've committed this.

        Show
        Eli Collins added a comment - Thanks Todd, I've committed this.
        Hide
        Todd Lipcon added a comment -

        +1

        Show
        Todd Lipcon added a comment - +1
        Todd Lipcon made changes -
        Summary Improve a blog recovery log Improve a block recovery log
        Hide
        Eli Collins added a comment -

        Unit tests are running, earlier run was clean. test-patch results follow. 4 findbugs warnings are HADOOP-7847, not this patch. Existing test coverage should be sufficient.

             [exec] 
             [exec] -1 overall.  
             [exec] 
             [exec]     +1 @author.  The patch does not contain any @author tags.
             [exec] 
             [exec]     -1 tests included.  The patch doesn't appear to include any new or modified tests.
             [exec]                         Please justify why no tests are needed for this patch.
             [exec] 
             [exec]     +1 javadoc.  The javadoc tool did not generate any warning messages.
             [exec] 
             [exec]     +1 javac.  The applied patch does not increase the total number of javac compiler warnings.
             [exec] 
             [exec]     -1 findbugs.  The patch appears to introduce 4 new Findbugs (version 1.3.9) warnings.
             [exec] 
        
        Show
        Eli Collins added a comment - Unit tests are running, earlier run was clean. test-patch results follow. 4 findbugs warnings are HADOOP-7847 , not this patch. Existing test coverage should be sufficient. [exec] [exec] -1 overall. [exec] [exec] +1 @author. The patch does not contain any @author tags. [exec] [exec] -1 tests included. The patch doesn't appear to include any new or modified tests. [exec] Please justify why no tests are needed for this patch. [exec] [exec] +1 javadoc. The javadoc tool did not generate any warning messages. [exec] [exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings. [exec] [exec] -1 findbugs. The patch appears to introduce 4 new Findbugs (version 1.3.9) warnings. [exec]
        Eli Collins made changes -
        Field Original Value New Value
        Attachment hdfs-2638-b1-1.patch [ 12506340 ]
        Hide
        Eli Collins added a comment -

        Patch attached.

        Show
        Eli Collins added a comment - Patch attached.
        Eli Collins created issue -

          People

          • Assignee:
            Eli Collins
            Reporter:
            Eli Collins
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development