[ https://issues.apache.org/jira/browse/PHOENIX-4350?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16242804#comment-16242804 ]
Josh Elser commented on PHOENIX-4350: ------------------------------------- {code} - byte[] splitPoint = s.getSplitPoint(); + Optional<byte[]> splitPoint = s.getSplitPoint(); long storeSize = s.getSize(); if (splitPoint != null && largestStoreSize < storeSize) { - splitPointFromLargestStore = splitPoint; + splitPointFromLargestStore = splitPoint.get(); {code} This will throw an NPE if the Region is not allowed to split (it {{hasReferences()}}). Need to change the conditional from {{splitPoint != null}} to {{splitPoint.isPresent()}}. Also, {{IndexRegionSplitPolicy}} extends {{IncreasingToUpperBoundRegionSplitPolicy}} which is marked as {{Private}}. Will file another issue to fix that later. I see we're still blocked on other compilation stuff, so I'm +1 if you want to commit after fixing the above, [~rajeshbabu] > Replace deprecated or changed Region methods with new APIs > ---------------------------------------------------------- > > Key: PHOENIX-4350 > URL: https://issues.apache.org/jira/browse/PHOENIX-4350 > Project: Phoenix > Issue Type: Sub-task > Reporter: Rajeshbabu Chintaguntla > Assignee: Rajeshbabu Chintaguntla > Labels: HBase-2.0 > Fix For: 4.14.0 > > Attachments: PHOENIX-4350.patch > > -- This message was sent by Atlassian JIRA (v6.4.14#64029)