[
https://issues.apache.org/jira/browse/PHOENIX-4376?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16252990#comment-16252990
]
Ankit Singhal commented on PHOENIX-4376:
----------------------------------------
bq. We should get rid of the ((HStore) store).getScanInfo(). Can you switch
ScanInfoUtil.isKeepDeleteCells to take a Store and have the method use
store.getColumnFamilyDescriptor().getKeepDeletedCells() instead?
Actually, this whole code needs to be removed and some alternative needs to
think for PHOENIX-4378.
Committed to 5.x-HBase-2.0 branch.
> Fix all compilation error for package org.apache.phoenix.schema.stats and
> org.apache.phoenix.coprocessor
> --------------------------------------------------------------------------------------------------------
>
> Key: PHOENIX-4376
> URL: https://issues.apache.org/jira/browse/PHOENIX-4376
> Project: Phoenix
> Issue Type: Sub-task
> Reporter: Ankit Singhal
> Assignee: Ankit Singhal
> Labels: HBase-2.0
> Fix For: 5.0.0
>
> Attachments: PHOENIX-4376.patch
>
>
> -- It includes updating the coprocessor hook with right definition.
> -- Changing HRegionInfo to RegionInfo.
> -- Cell related changes.
> Like
> {code}
> - byte[] viewConstant = viewConstantKv == null ? null :
> viewConstantKv.getValue();
> + byte[] viewConstant = viewConstantKv == null ? null :
> CellUtil.cloneValue(viewConstantKv);
> - Cell viewStatementCell = new KeyValue(cell.getRow(),
> cell.getFamily(), VIEW_STATEMENT_BYTES,
> + Cell viewStatementCell = new
> KeyValue(CellUtil.cloneRow(cell), CellUtil.cloneFamily(cell),
> VIEW_STATEMENT_BYTES,
> {code}
> -- Update scan
> {code}
> - scan.setStartRow(HConstants.EMPTY_START_ROW);
> - scan.setStopRow(HConstants.EMPTY_END_ROW);
> + scan.withStartRow(HConstants.EMPTY_START_ROW);
> + scan.withStopRow(HConstants.EMPTY_END_ROW);
> {code}
> -- preSplit hook needs to be removed and fixed in PHOENIX-4352.
> -- Requires a separate fix of PHOENIX-4277 and PHOENIX-3081 for HBase 2.0
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
