Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Date: Wed, 4 May 2016 17:53:13 +0000 (UTC)
From: "Ted Yu (JIRA)" <jira@apache.org>
To: issues@hbase.apache.org
Message-ID: <JIRA.12964670.1462282919000.110903.1462384393560@Atlassian.JIRA>
In-Reply-To: <JIRA.12964670.1462282919000@Atlassian.JIRA>
References: <JIRA.12964670.1462282919000@Atlassian.JIRA> <JIRA.12964670.1462282919977@arcas>
Subject: [jira] [Updated] (HBASE-15759) RegionObserver.preStoreScannerOpen()
 doesn't have acces to current readpoint
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Wed, 04 May 2016 17:53:15 -0000


     [ https://issues.apache.org/jira/browse/HBASE-15759?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ted Yu updated HBASE-15759:
---------------------------
    Attachment: 15759.v3.patch

Patch v3 addresses failed tests.

> RegionObserver.preStoreScannerOpen() doesn't have acces to current readpoint
> ----------------------------------------------------------------------------
>
>                 Key: HBASE-15759
>                 URL: https://issues.apache.org/jira/browse/HBASE-15759
>             Project: HBase
>          Issue Type: Improvement
>          Components: Coprocessors
>            Reporter: Marek Srank
>            Assignee: Ted Yu
>            Priority: Minor
>              Labels: coprocessors
>         Attachments: 15759.v1.patch, 15759.v2.patch, 15759.v3.patch
>
>
> RegionObserver.preStoreScannerOpen() doesn't have acces to current readpoint. This is crucial e.g. when creating a new StoreScanner.
> Although it is possible to obtain the readpoint via following workaround:
> ((HStore)store).getHRegion().getReadpoint(IsolationLevel.READ_COMMITTED)
> ...it presumes some knowledge of HBase internals and is not a clear option for user.
> The problem was introduced in HBASE-9754, where readpoint was introduced as an argument to StoreScanner constructor, but wasn't passed to the preStoreScannerOpen() hook in HStore.getScanner().
> In our case it led to bug, when we were incorrectly using smallestReadpoint() instead, which made coprocessor missing data in some cases [1].
> __________
> [1] http://mail-archives.apache.org/mod_mbox/hbase-dev/201604.mbox/%3CCAOUjMkwptCGacQ0d-4yOinDVBT%2Bbi2NOZee0fSbq%2Bk1R6V4-aw%40mail.gmail.com%3E


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