[
https://issues.apache.org/jira/browse/HBASE-12566?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14250371#comment-14250371
]
Andrew Purtell commented on HBASE-12566:
----------------------------------------
If HBase makes HRegion and SplitTransaction LimitedPrivate, this promotes all
low level functions they are concerned with: all low level aspects of regions
(HRegion), filesystem layouts, particulars on how assignments are reported to
the master, particulars on how meta table entries are encoded. This is
tantamount to importing the Phoenix code into HBase proper. (I suppose that's
an option.)
I think Phoenix-ers and HBasers need to meet and hash out a supportable
LimitedPrivate interface for what local indexes need.
> HRegion should have an InterfaceAudience of LimitedPrivate(PHOENIX)
> -------------------------------------------------------------------
>
> Key: HBASE-12566
> URL: https://issues.apache.org/jira/browse/HBASE-12566
> Project: HBase
> Issue Type: Bug
> Reporter: Andrew Purtell
> Labels: Phoenix
>
> I've discovered after HBASE-12550 that Phoenix has a class that was broken by
> a change to a package scoped method in HRegion:
> {code}
> diff --git
> a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
> b/hbase-server/src/main/java/org/apache/hadoop/hbase/reg
> index 39a9fdc..3377e6b 100644
> ---
> a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
> +++
> b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
> @@ -4628,11 +4628,12 @@ public class HRegion implements HeapSize { // ,
> Writable{
> /**
> * Create a daughter region from given a temp directory with the region
> data.
> * @param hri Spec. for daughter region to open.
> + * @param expectedReferenceFileCount
> * @throws IOException
> */
> - HRegion createDaughterRegionFromSplits(final HRegionInfo hri) throws
> IOException {
> + HRegion createDaughterRegionFromSplits(final HRegionInfo hri, int
> expectedReferenceFileCount) throws IOException {
> // Move the files from the temporary .splits to the final /table/region
> directory
> - fs.commitDaughterRegion(hri);
> + fs.commitDaughterRegion(hri, expectedReferenceFileCount);
> {code}
> We should change the HRegion InterfaceAudience to LimitedPrivate(COPROC,
> PHOENIX).
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
