[ 
https://issues.apache.org/jira/browse/HBASE-16962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15646321#comment-15646321
 ] 

Anoop Sam John commented on HBASE-16962:
----------------------------------------

For preStoreScannerOpen, we pass the readPnt already
{code}
@Deprecated
  KeyValueScanner preStoreScannerOpen(final 
ObserverContext<RegionCoprocessorEnvironment> c,
      final Store store, final Scan scan, final NavigableSet<byte[]> targetCols,
      final KeyValueScanner s) throws IOException;

KeyValueScanner preStoreScannerOpen(final 
ObserverContext<RegionCoprocessorEnvironment> c,
      final Store store, final Scan scan, final NavigableSet<byte[]> targetCols,
      final KeyValueScanner s, final long readPt) throws IOException;
{code}
Am +1 for passing readPnt for the mentioned hook here.  With readPnt only it 
make full sense of pre hook.  There might be other candidates also. Like 
preFlushScannerOpen..   
May be we can stick to only this method here. Ur call.  We wl have to deprecate 
old API and add new one.

> Add readPoint to preCompactScannerOpen() API
> --------------------------------------------
>
>                 Key: HBASE-16962
>                 URL: https://issues.apache.org/jira/browse/HBASE-16962
>             Project: HBase
>          Issue Type: Bug
>    Affects Versions: 2.0.0
>            Reporter: Thiruvel Thirumoolan
>            Assignee: Thiruvel Thirumoolan
>         Attachments: HBASE-16962.master.001.patch, HBASE-16962.rough.patch
>
>
> Similar to HBASE-15759, I would like to add readPoint to the 
> preCompactScannerOpen() API.
> I have a CP where I create a StoreScanner() as part of the 
> preCompactScannerOpen() API. I need the readpoint which was obtained in 
> Compactor.compact() method to be consistent.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to