[ https://issues.apache.org/jira/browse/HBASE-18049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16011621#comment-16011621 ]
Anoop Sam John commented on HBASE-18049: ---------------------------------------- In above lines we have {code} if ((e instanceof FileNotFoundException) || (e.getCause() instanceof FileNotFoundException)) { {code} So here in new change as well, we need getCause() check? Just asking. I did not check the flow to see how the Exception stack will be possibly be. Else looks ok. > It is not necessary to re-open the region when MOB files cannot be found > ------------------------------------------------------------------------ > > Key: HBASE-18049 > URL: https://issues.apache.org/jira/browse/HBASE-18049 > Project: HBase > Issue Type: Bug > Components: Scanners > Reporter: Jingcheng Du > Assignee: Jingcheng Du > Fix For: 2.0.0 > > Attachments: HBASE-18049.patch > > > In HBASE-17712, we try to re-open the region when store files cannot be > found. This is useful for store files in a region, but is not necessary when > the MOB files cannot be found, because the store files in a region only > contain the references to the MOB files and a re-open of a region doesn't > help the lost MOB files. > In this JIRA, we will directly throw DNRIOE only when the MOB files are not > found in {{MobStoreScanner}} and {{ReversedMobStoreScanner}}. Other logics > keep the same. -- This message was sent by Atlassian JIRA (v6.3.15#6346)