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

    • Sub-task
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 0.99.2
    • regionserver, Scanners
    • None
    • 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

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

            Dates

              Created:
              Updated:
              Resolved: