Details

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

      Description

      TestListFiles assumes a particular order of the files returned by the directory iterator. There's no such guarantee made by the underlying API, so the test fails on some hosts.

        Activity

        Hide
        Tsz Wo Nicholas Sze added a comment -

        +1 patch looks good.

        Show
        Tsz Wo Nicholas Sze added a comment - +1 patch looks good.
        Hide
        Giridharan Kesavan added a comment -

        re-submiting to hudson for testing hudson patch tst process.

        Show
        Giridharan Kesavan added a comment - re-submiting to hudson for testing hudson patch tst process.
        Hide
        Tom White added a comment -

        Result of test-patch:

        [exec] +1 overall.  
        [exec] 
        [exec]     +1 @author.  The patch does not contain any @author tags.
        [exec] 
        [exec]     +1 tests included.  The patch appears to include 3 new or modified tests.
        [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 does not introduce any new Findbugs warnings.
        [exec] 
        [exec]     +1 release audit.  The applied patch does not increase the total number of release audit warnings.
        [exec] 
        [exec]     +1 system tests framework.  The patch passed system tests framework compile.
        [exec] 
        
        Show
        Tom White added a comment - Result of test-patch: [exec] +1 overall. [exec] [exec] +1 @author. The patch does not contain any @author tags. [exec] [exec] +1 tests included. The patch appears to include 3 new or modified tests. [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 does not introduce any new Findbugs warnings. [exec] [exec] +1 release audit. The applied patch does not increase the total number of release audit warnings. [exec] [exec] +1 system tests framework. The patch passed system tests framework compile. [exec]
        Hide
        Tom White added a comment -

        I've just committed this. Thanks, Todd!

        Show
        Tom White added a comment - I've just committed this. Thanks, Todd!
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Common-trunk-Commit #395 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk-Commit/395/)

        Show
        Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #395 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk-Commit/395/ )
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Common-trunk #489 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk/489/)

        Show
        Hudson added a comment - Integrated in Hadoop-Common-trunk #489 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk/489/ )

          People

          • Assignee:
            Todd Lipcon
            Reporter:
            Todd Lipcon
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development