You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hbase.apache.org by "Duo Zhang (JIRA)" <ji...@apache.org> on 2017/03/01 07:20:45 UTC

[jira] [Created] (HBASE-17712) Remove/Simplify the logic of RegionScannerImpl.handleFileNotFound

Duo Zhang created HBASE-17712:
---------------------------------

             Summary: Remove/Simplify the logic of RegionScannerImpl.handleFileNotFound
                 Key: HBASE-17712
                 URL: https://issues.apache.org/jira/browse/HBASE-17712
             Project: HBase
          Issue Type: Bug
    Affects Versions: 2.0.0, 1.4.0
            Reporter: Duo Zhang
             Fix For: 2.0.0, 1.4.0


It is introduced in HBASE-13651 and the logic became much more complicated after HBASE-16304 due to a dead lock issue. It is really tough as sequence id is involved in and the method we called is used to
serve secondary replica originally which does not handle write.

In fact, in 1.x release, the problem described in HBASE-13651 is gone. Now we will write a compaction marker to WAL before deleting the compacted files. We can only consider a RS as dead after its WAL files are all closed so if the region has already been reassigned the compaction will fail as we can not write out the compaction marker.

So theoretically, if we still hit FileNotFound exception, it should be a critical bug which means we may loss data. I do not think it is a good idea to just eat the exception and refresh store files. Or even if we want to do this, we can just refresh store files without dropping memstore contents. This will also simplify the logic a lot.

Suggestions are welcomed.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)