Details

    • Type: Test Test
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.21.1, 0.22.0
    • Fix Version/s: 0.21.1, 0.22.0
    • Component/s: record
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      The test for the ByteWritable comparator bug (see HADOOP-6928), which is already fixed in 0.21.

      1. h-6934.patch
        2 kB
        Johannes Zillmann
      2. h-6934-2.patch
        2 kB
        Eli Collins

        Issue Links

          Activity

          Johannes Zillmann created issue -
          Hide
          Johannes Zillmann added a comment -

          Patch h-6934.patch with the test attached.

          Show
          Johannes Zillmann added a comment - Patch h-6934.patch with the test attached.
          Johannes Zillmann made changes -
          Field Original Value New Value
          Attachment h-6934.patch [ 12453357 ]
          Johannes Zillmann made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Giridharan Kesavan added a comment -

          re-submiting to test hudson patch testing

          Show
          Giridharan Kesavan added a comment - re-submiting to test hudson patch testing
          Giridharan Kesavan made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Eli Collins made changes -
          Link This issue relates to HADOOP-6928 [ HADOOP-6928 ]
          Hide
          Eli Collins added a comment -

          +1 (same test from HADOOP-6934)

          Updated patch to be junit-4 style.

          Show
          Eli Collins added a comment - +1 (same test from HADOOP-6934 ) Updated patch to be junit-4 style.
          Eli Collins made changes -
          Attachment h-6934-2.patch [ 12453649 ]
          Eli Collins made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hadoop Flags [Reviewed]
          Affects Version/s 0.22.0 [ 12314296 ]
          Fix Version/s 0.22.0 [ 12314296 ]
          Hide
          Eli Collins added a comment -

          I've commited this (it just adds a test and that test passes). Thanks Johannes!

          Show
          Eli Collins added a comment - I've commited this (it just adds a test and that test passes). Thanks Johannes!
          Eli Collins made changes -
          Assignee Johannes Zillmann [ oae ]
          Fix Version/s 0.21.1 [ 12315270 ]
          Affects Version/s 0.21.0 [ 12313563 ]
          Eli Collins made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk-Commit #372 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk-Commit/372/)
          HADOOP-6934. Test for ByteWritable comparator. Contributed by Johannes Zillmann.

          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #372 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk-Commit/372/ ) HADOOP-6934 . Test for ByteWritable comparator. Contributed by Johannes Zillmann.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk #441 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk/441/)
          HADOOP-6934. Test for ByteWritable comparator. Contributed by Johannes Zillmann.

          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk #441 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk/441/ ) HADOOP-6934 . Test for ByteWritable comparator. Contributed by Johannes Zillmann.
          Hide
          Owen O'Malley added a comment -

          Shouldn't this be fixed in the 0.21 branch too?

          Show
          Owen O'Malley added a comment - Shouldn't this be fixed in the 0.21 branch too?
          Hide
          Eli Collins added a comment -

          Yes, I'll merge it.

          Show
          Eli Collins added a comment - Yes, I'll merge it.
          Eli Collins made changes -
          Fix Version/s 0.21.1 [ 12315270 ]
          Affects Version/s 0.21.1 [ 12315270 ]
          Hide
          Eli Collins added a comment -

          I merged this into 21.

          Show
          Eli Collins added a comment - I merged this into 21.
          Konstantin Shvachko made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Patch Available Patch Available Open Open
          3d 3h 2m 1 Giridharan Kesavan 31/Aug/10 20:58
          Open Open Patch Available Patch Available
          1d 5h 9m 2 Eli Collins 02/Sep/10 02:00
          Patch Available Patch Available Resolved Resolved
          4h 28m 1 Eli Collins 02/Sep/10 06:28
          Resolved Resolved Closed Closed
          465d 23h 51m 1 Konstantin Shvachko 12/Dec/11 06:19

            People

            • Assignee:
              Johannes Zillmann
              Reporter:
              Johannes Zillmann
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development