[ https://issues.apache.org/jira/browse/HBASE-12566?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14252207#comment-14252207 ]
Andrew Purtell commented on HBASE-12566: ---------------------------------------- Yes, but in that case this type is used by the coprocessor API so we need to change the annotation or provide a replacement copacetic type in 1.0. > 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)