[HBASE-19607] throw the exception if memstore is incompetent to do the deep clone for cell

• Type: Bug
• Status: Open
• Priority: Trivial
• Resolution: Unresolved
• Assignee: Yung-An He
• Reporter: Chia-Ping Tsai
• Created: 24/Dec/17 10:52
• Updated: 12/Jan/18 11:28

Description

We must to clone the cell since the cell is backed by the reusable byte array. Also, we assume all cells passed to AbstractMemStore is of ExtendedCell. Not only is the type check unnecessary, but it also confuses the readers.

AbstractMemStore.java

  private static Cell deepCopyIfNeeded(Cell cell) {
    if (cell instanceof ExtendedCell) {
      return ((ExtendedCell) cell).deepClone();
    }
    return cell;
  }