Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-11425 Cell/DBB end-to-end on the read-path
  3. HBASE-12313

Redo the hfile index length optimization so cell-based rather than serialized KV key

    XMLWordPrintableJSON

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.99.2
    • Component/s: regionserver, Scanners
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Trying to remove API that returns the 'key' of a KV serialized into a byte array is thorny.

      I tried to move over the first and last key serializations and the hfile index entries to be cell but patch was turning massive. Here is a smaller patch that just redoes the optimization that tries to find 'short' midpoints between last key of last block and first key of next block so it is Cell-based rather than byte array based (presuming Keys serialized in a certain way). Adds unit tests which we didn't have before.

      Also remove CellKey. Not needed... at least not yet. Its just utility for toString.

        Attachments

        1. 0001-HBASE-12313-Redo-the-hfile-index-length-optimization.patch
          81 kB
          Michael Stack
        2. 0001-HBASE-12313-Redo-the-hfile-index-length-optimization.patch
          81 kB
          Michael Stack
        3. 0001-HBASE-12313-Redo-the-hfile-index-length-optimization.patch
          78 kB
          Michael Stack
        4. 0001-HBASE-12313-Redo-the-hfile-index-length-optimization.patch
          78 kB
          Michael Stack
        5. 0001-HBASE-12313-Redo-the-hfile-index-length-optimization.patch
          78 kB
          Michael Stack
        6. 12313v10.txt
          81 kB
          Michael Stack
        7. 12313v5.txt
          78 kB
          Michael Stack
        8. 12313v6.txt
          81 kB
          Michael Stack
        9. 12313v8.txt
          81 kB
          Michael Stack

          Activity

            People

            • Assignee:
              stack Michael Stack
              Reporter:
              stack Michael Stack
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: