Hadoop Common
  1. Hadoop Common
  2. HADOOP-8323

Revert HADOOP-7940 and improve javadocs and test for Text.clear()

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0.0-alpha
    • Fix Version/s: 2.0.2-alpha
    • Component/s: io
    • Labels:

      Description

      Per Jim Donofrio's comments on HADOOP-7940, we should revert it as it has caused a performance regression (for scenarios where Text is reused, popular in MR).

      The clear() works as intended, as the API also offers a current length API.

      1. HADOOP-8323.patch
        3 kB
        Harsh J
      2. HADOOP-8323.patch
        3 kB
        Harsh J
      3. HADOOP-8323.patch
        3 kB
        Harsh J

        Issue Links

          Activity

          Arun C Murthy made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Arun C Murthy made changes -
          Fix Version/s 2.0.2-alpha [ 12322473 ]
          Fix Version/s 2.1.0-alpha [ 12321441 ]
          Harsh J made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Target Version/s 2.0.1-alpha, 3.0.0 [ 12321441, 12320357 ]
          Fix Version/s 2.0.1-alpha [ 12321441 ]
          Resolution Fixed [ 1 ]
          Harsh J made changes -
          Target Version/s 2.0.0-alpha, 3.0.0 [ 12320352, 12320357 ] 2.0.1-alpha, 3.0.0 [ 12321441, 12320357 ]
          Harsh J made changes -
          Issue Type Bug [ 1 ] Improvement [ 4 ]
          Harsh J made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Target Version/s 3.0.0, 2.0.0 [ 12320357, 12320352 ] 2.0.0, 3.0.0 [ 12320352, 12320357 ]
          Harsh J made changes -
          Attachment HADOOP-8323.patch [ 12525770 ]
          Harsh J made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Target Version/s 3.0.0, 2.0.0 [ 12320357, 12320352 ] 2.0.0, 3.0.0 [ 12320352, 12320357 ]
          Harsh J made changes -
          Attachment HADOOP-8323.patch [ 12525019 ]
          Harsh J made changes -
          Summary Revert HADOOP-7940 Revert HADOOP-7940 and improve javadocs and test for Text.clear()
          Labels performance
          Target Version/s 3.0.0, 2.0.0 [ 12320357, 12320352 ] 2.0.0, 3.0.0 [ 12320352, 12320357 ]
          Harsh J made changes -
          Status Reopened [ 4 ] Patch Available [ 10002 ]
          Target Version/s 2.0.0, 3.0.0 [ 12320352, 12320357 ]
          Harsh J made changes -
          Attachment HADOOP-8323.patch [ 12524964 ]
          Harsh J made changes -
          Resolution Won't Fix [ 2 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Harsh J made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Target Version/s 3.0.0, 2.0.0 [ 12320357, 12320352 ]
          Resolution Won't Fix [ 2 ]
          Harsh J made changes -
          Field Original Value New Value
          Link This issue supercedes HADOOP-7940 [ HADOOP-7940 ]
          Harsh J created issue -

            People

            • Assignee:
              Harsh J
              Reporter:
              Harsh J
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development