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

KeyValue to Cell Convert in WALEdit APIs

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 0.99.0
    • wal
    • None
    • Incompatible change, Reviewed
    • Hide
      The KeyValue based APIs in WALEdit is been removed and replaced with Cell based APIs
      void add(KeyValue) -> void add(Cell)
      ArrayList<KeyValue> getKeyValues() -> ArrayList<Cell> getCells()
      Show
      The KeyValue based APIs in WALEdit is been removed and replaced with Cell based APIs void add(KeyValue) -> void add(Cell) ArrayList<KeyValue> getKeyValues() -> ArrayList<Cell> getCells()

    Description

      In almost all other main interface class/APIs we have changed KeyValue to Cell. But missing in WALEdit. This is public marked for Replication (Well it should be for CP also)
      These 2 APIs deal with KVs
      add(KeyValue kv)
      ArrayList<KeyValue> getKeyValues()

      Suggest deprecate them and add for 0.98
      add(Cell kv)
      List<Cell> getCells()

      And just replace from 1.0

      Attachments

        1. HBASE-11805_0.98_addendum.patch
          30 kB
          Anoop Sam John
        2. HBASE-11805_0.98_V2.patch
          56 kB
          Anoop Sam John
        3. HBASE-11805_0.98.patch
          54 kB
          Anoop Sam John
        4. HBASE-11805_0.99.patch
          70 kB
          Anoop Sam John
        5. HBASE-11805_V2.patch
          70 kB
          Anoop Sam John
        6. HBASE-11805_V3.patch
          72 kB
          Anoop Sam John
        7. HBASE-11805.patch
          69 kB
          Anoop Sam John

        Issue Links

          Activity

            People

              anoop.hbase Anoop Sam John
              anoop.hbase Anoop Sam John
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: