Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-24888 API cleanup for 3.0.0
  3. HBASE-28684

Remove CellWrapper and use ExtendedCell internally in client side data structure

    XMLWordPrintableJSON

Details

    • Incompatible change, Reviewed
    • Hide
      Removed CellWrapper so now you can only add Cells which are ExtendedCells to all the data structures in HBase like Mutation, Get, etc.
      Usually this will not introduce any problem because the Cells returned by CellBuilder will be ExtendedCells. It is not recommended to have your own Cell implementation as there are some internal status which are not exposed through the Cell interface but we highly depend on these status inside HBase implementation.
      For coprocessor users, if you really need to have your own Cell implementation, please implement ExtendedCell, but since it is IA.Private, take your own risk. You can report to our mailing list about your usage and we will see how to better support it.
      Show
      Removed CellWrapper so now you can only add Cells which are ExtendedCells to all the data structures in HBase like Mutation, Get, etc. Usually this will not introduce any problem because the Cells returned by CellBuilder will be ExtendedCells. It is not recommended to have your own Cell implementation as there are some internal status which are not exposed through the Cell interface but we highly depend on these status inside HBase implementation. For coprocessor users, if you really need to have your own Cell implementation, please implement ExtendedCell, but since it is IA.Private, take your own risk. You can report to our mailing list about your usage and we will see how to better support it.

    Description

      In general, all Cells in HBase are ExtendedCells, we introduce Cell interface is only for preventing user to call some methods which damage the system.

      So I think we should have internal methods which can get ExtendedCell from the client side data structures so we do not need to cast everywhere.

      Attachments

        Issue Links

          Activity

            People

              zhangduo Duo Zhang
              zhangduo Duo Zhang
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: