Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-14227

Fold special cased MOB APIs into existing APIs

    XMLWordPrintableJSON

    Details

    • Type: Task
    • Status: Resolved
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0.0
    • Fix Version/s: 2.0.0
    • Component/s: mob
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      There are a number of APIs that came in with MOB that are not new actions for HBase, simply new actions for a MOB implementation:

      • compactMob
      • compactMobs
      • majorCompactMob
      • majorCompactMobs
      • getMobCompactionState

      And in HBaseAdmin:

      • validateMobColumnFamily

      Remove these special cases from the Admin API where possible by folding them into existing APIs.

      We definitely don't need one method for a singleton and another for collections.

      Ideally we will not have any APIs named *Mob when finished, whether MOBs are in use on a table or not should be largely an internal detail. Exposing as schema option would be fine, this conforms to existing practice for other features.

      Marking critical because I think removing the *Mob special cased APIs should be a precondition for release of this feature either in 2.0 or as a backport.

        Attachments

        1. HBASE-14227.patch
          17 kB
          Jingcheng Du
        2. HBASE-14227_v7.patch
          36 kB
          Heng Chen
        3. HBASE-14227_v6.patch
          36 kB
          Heng Chen
        4. HBASE-14227_v5.patch
          27 kB
          Heng Chen
        5. HBASE-14227_v5.patch
          27 kB
          Sean Busbey
        6. HBASE-14227_v4.patch
          27 kB
          Heng Chen
        7. HBASE-14227_v3.patch
          27 kB
          Heng Chen
        8. HBASE-14227_v2.patch
          27 kB
          Heng Chen
        9. HBASE-14227_v1.patch
          25 kB
          Heng Chen

          Issue Links

            Activity

              People

              • Assignee:
                chenheng Heng Chen
                Reporter:
                apurtell Andrew Kyle Purtell
              • Votes:
                0 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: