http://git-wip-us.apache.org/repos/asf/hbase-site/blob/b7626a4c/devapidocs/src-html/org/apache/hadoop/hbase/replication/regionserver/WALEntryStream.html
----------------------------------------------------------------------
diff --git 
a/devapidocs/src-html/org/apache/hadoop/hbase/replication/regionserver/WALEntryStream.html
 
b/devapidocs/src-html/org/apache/hadoop/hbase/replication/regionserver/WALEntryStream.html
index 745d53a..613d163 100644
--- 
a/devapidocs/src-html/org/apache/hadoop/hbase/replication/regionserver/WALEntryStream.html
+++ 
b/devapidocs/src-html/org/apache/hadoop/hbase/replication/regionserver/WALEntryStream.html
@@ -241,190 +241,192 @@
 <span class="sourceLineNo">233</span>      if (trailerSize &lt; 0) {<a 
name="line.233"></a>
 <span class="sourceLineNo">234</span>        if (currentPosition &lt; 
stat.getLen()) {<a name="line.234"></a>
 <span class="sourceLineNo">235</span>          final long skippedBytes = 
stat.getLen() - currentPosition;<a name="line.235"></a>
-<span class="sourceLineNo">236</span>          LOG.info("Reached the end of 
WAL file '" + currentPath<a name="line.236"></a>
-<span class="sourceLineNo">237</span>              + "'. It was not closed 
cleanly, so we did not parse " + skippedBytes<a name="line.237"></a>
-<span class="sourceLineNo">238</span>              + " bytes of data.");<a 
name="line.238"></a>
-<span class="sourceLineNo">239</span>          
metrics.incrUncleanlyClosedWALs();<a name="line.239"></a>
-<span class="sourceLineNo">240</span>          
metrics.incrBytesSkippedInUncleanlyClosedWALs(skippedBytes);<a 
name="line.240"></a>
-<span class="sourceLineNo">241</span>        }<a name="line.241"></a>
-<span class="sourceLineNo">242</span>      } else if (currentPosition + 
trailerSize &lt; stat.getLen()) {<a name="line.242"></a>
-<span class="sourceLineNo">243</span>        LOG.warn("Processing end of WAL 
file '" + currentPath + "'. At position " + currentPosition<a 
name="line.243"></a>
-<span class="sourceLineNo">244</span>            + ", which is too far away 
from reported file length " + stat.getLen()<a name="line.244"></a>
-<span class="sourceLineNo">245</span>            + ". Restarting WAL reading 
(see HBASE-15983 for details). " + getCurrentPathStat());<a name="line.245"></a>
-<span class="sourceLineNo">246</span>        setPosition(0);<a 
name="line.246"></a>
-<span class="sourceLineNo">247</span>        resetReader();<a 
name="line.247"></a>
-<span class="sourceLineNo">248</span>        
metrics.incrRestartedWALReading();<a name="line.248"></a>
-<span class="sourceLineNo">249</span>        
metrics.incrRepeatedFileBytes(currentPosition);<a name="line.249"></a>
-<span class="sourceLineNo">250</span>        return false;<a 
name="line.250"></a>
-<span class="sourceLineNo">251</span>      }<a name="line.251"></a>
-<span class="sourceLineNo">252</span>    }<a name="line.252"></a>
-<span class="sourceLineNo">253</span>    if (LOG.isTraceEnabled()) {<a 
name="line.253"></a>
-<span class="sourceLineNo">254</span>      LOG.trace("Reached the end of log " 
+ this.currentPath + ", and the length of the file is "<a name="line.254"></a>
-<span class="sourceLineNo">255</span>          + (stat == null ? "N/A" : 
stat.getLen()));<a name="line.255"></a>
-<span class="sourceLineNo">256</span>    }<a name="line.256"></a>
-<span class="sourceLineNo">257</span>    metrics.incrCompletedWAL();<a 
name="line.257"></a>
-<span class="sourceLineNo">258</span>    return true;<a name="line.258"></a>
-<span class="sourceLineNo">259</span>  }<a name="line.259"></a>
-<span class="sourceLineNo">260</span><a name="line.260"></a>
-<span class="sourceLineNo">261</span>  private void dequeueCurrentLog() throws 
IOException {<a name="line.261"></a>
-<span class="sourceLineNo">262</span>    if (LOG.isDebugEnabled()) {<a 
name="line.262"></a>
-<span class="sourceLineNo">263</span>      LOG.debug("Reached the end of log " 
+ currentPath);<a name="line.263"></a>
-<span class="sourceLineNo">264</span>    }<a name="line.264"></a>
-<span class="sourceLineNo">265</span>    closeReader();<a name="line.265"></a>
-<span class="sourceLineNo">266</span>    logQueue.remove();<a 
name="line.266"></a>
-<span class="sourceLineNo">267</span>    setPosition(0);<a name="line.267"></a>
-<span class="sourceLineNo">268</span>    metrics.decrSizeOfLogQueue();<a 
name="line.268"></a>
-<span class="sourceLineNo">269</span>  }<a name="line.269"></a>
-<span class="sourceLineNo">270</span><a name="line.270"></a>
-<span class="sourceLineNo">271</span>  private void 
readNextEntryAndSetPosition() throws IOException {<a name="line.271"></a>
-<span class="sourceLineNo">272</span>    Entry readEntry = reader.next();<a 
name="line.272"></a>
-<span class="sourceLineNo">273</span>    long readerPos = 
reader.getPosition();<a name="line.273"></a>
-<span class="sourceLineNo">274</span>    if (readEntry != null) {<a 
name="line.274"></a>
-<span class="sourceLineNo">275</span>      metrics.incrLogEditsRead();<a 
name="line.275"></a>
-<span class="sourceLineNo">276</span>      
metrics.incrLogReadInBytes(readerPos - currentPosition);<a name="line.276"></a>
-<span class="sourceLineNo">277</span>    }<a name="line.277"></a>
-<span class="sourceLineNo">278</span>    currentEntry = readEntry; // could be 
null<a name="line.278"></a>
-<span class="sourceLineNo">279</span>    setPosition(readerPos);<a 
name="line.279"></a>
-<span class="sourceLineNo">280</span>  }<a name="line.280"></a>
-<span class="sourceLineNo">281</span><a name="line.281"></a>
-<span class="sourceLineNo">282</span>  private void closeReader() throws 
IOException {<a name="line.282"></a>
-<span class="sourceLineNo">283</span>    if (reader != null) {<a 
name="line.283"></a>
-<span class="sourceLineNo">284</span>      reader.close();<a 
name="line.284"></a>
-<span class="sourceLineNo">285</span>      reader = null;<a 
name="line.285"></a>
-<span class="sourceLineNo">286</span>    }<a name="line.286"></a>
-<span class="sourceLineNo">287</span>  }<a name="line.287"></a>
-<span class="sourceLineNo">288</span><a name="line.288"></a>
-<span class="sourceLineNo">289</span>  // if we don't have a reader, open a 
reader on the next log<a name="line.289"></a>
-<span class="sourceLineNo">290</span>  private boolean checkReader() throws 
IOException {<a name="line.290"></a>
-<span class="sourceLineNo">291</span>    if (reader == null) {<a 
name="line.291"></a>
-<span class="sourceLineNo">292</span>      return openNextLog();<a 
name="line.292"></a>
-<span class="sourceLineNo">293</span>    }<a name="line.293"></a>
-<span class="sourceLineNo">294</span>    return true;<a name="line.294"></a>
-<span class="sourceLineNo">295</span>  }<a name="line.295"></a>
-<span class="sourceLineNo">296</span><a name="line.296"></a>
-<span class="sourceLineNo">297</span>  // open a reader on the next log in 
queue<a name="line.297"></a>
-<span class="sourceLineNo">298</span>  private boolean openNextLog() throws 
IOException {<a name="line.298"></a>
-<span class="sourceLineNo">299</span>    Path nextPath = logQueue.peek();<a 
name="line.299"></a>
-<span class="sourceLineNo">300</span>    if (nextPath != null) {<a 
name="line.300"></a>
-<span class="sourceLineNo">301</span>      openReader(nextPath);<a 
name="line.301"></a>
-<span class="sourceLineNo">302</span>      if (reader != null) return true;<a 
name="line.302"></a>
-<span class="sourceLineNo">303</span>    }<a name="line.303"></a>
-<span class="sourceLineNo">304</span>    return false;<a name="line.304"></a>
-<span class="sourceLineNo">305</span>  }<a name="line.305"></a>
-<span class="sourceLineNo">306</span><a name="line.306"></a>
-<span class="sourceLineNo">307</span>  private Path getArchivedLog(Path path) 
throws IOException {<a name="line.307"></a>
-<span class="sourceLineNo">308</span>    Path rootDir = 
FSUtils.getRootDir(conf);<a name="line.308"></a>
-<span class="sourceLineNo">309</span>    Path oldLogDir = new Path(rootDir, 
HConstants.HREGION_OLDLOGDIR_NAME);<a name="line.309"></a>
-<span class="sourceLineNo">310</span>    Path archivedLogLocation = new 
Path(oldLogDir, path.getName());<a name="line.310"></a>
-<span class="sourceLineNo">311</span>    if (fs.exists(archivedLogLocation)) 
{<a name="line.311"></a>
-<span class="sourceLineNo">312</span>      LOG.info("Log " + path + " was 
moved to " + archivedLogLocation);<a name="line.312"></a>
-<span class="sourceLineNo">313</span>      return archivedLogLocation;<a 
name="line.313"></a>
-<span class="sourceLineNo">314</span>    } else {<a name="line.314"></a>
-<span class="sourceLineNo">315</span>      LOG.error("Couldn't locate log: " + 
path);<a name="line.315"></a>
-<span class="sourceLineNo">316</span>      return path;<a name="line.316"></a>
-<span class="sourceLineNo">317</span>    }<a name="line.317"></a>
-<span class="sourceLineNo">318</span>  }<a name="line.318"></a>
-<span class="sourceLineNo">319</span><a name="line.319"></a>
-<span class="sourceLineNo">320</span>  private void handleFileNotFound(Path 
path, FileNotFoundException fnfe) throws IOException {<a name="line.320"></a>
-<span class="sourceLineNo">321</span>    // If the log was archived, continue 
reading from there<a name="line.321"></a>
-<span class="sourceLineNo">322</span>    Path archivedLog = 
getArchivedLog(path);<a name="line.322"></a>
-<span class="sourceLineNo">323</span>    if (!path.equals(archivedLog)) {<a 
name="line.323"></a>
-<span class="sourceLineNo">324</span>      openReader(archivedLog);<a 
name="line.324"></a>
-<span class="sourceLineNo">325</span>    } else {<a name="line.325"></a>
-<span class="sourceLineNo">326</span>      throw fnfe;<a name="line.326"></a>
-<span class="sourceLineNo">327</span>    }<a name="line.327"></a>
-<span class="sourceLineNo">328</span>  }<a name="line.328"></a>
-<span class="sourceLineNo">329</span>  private void openReader(Path path) 
throws IOException {<a name="line.329"></a>
-<span class="sourceLineNo">330</span>    try {<a name="line.330"></a>
-<span class="sourceLineNo">331</span>      // Detect if this is a new file, if 
so get a new reader else<a name="line.331"></a>
-<span class="sourceLineNo">332</span>      // reset the current reader so that 
we see the new data<a name="line.332"></a>
-<span class="sourceLineNo">333</span>      if (reader == null || 
!getCurrentPath().equals(path)) {<a name="line.333"></a>
-<span class="sourceLineNo">334</span>        closeReader();<a 
name="line.334"></a>
-<span class="sourceLineNo">335</span>        reader = 
WALFactory.createReader(fs, path, conf);<a name="line.335"></a>
-<span class="sourceLineNo">336</span>        seek();<a name="line.336"></a>
-<span class="sourceLineNo">337</span>        setCurrentPath(path);<a 
name="line.337"></a>
-<span class="sourceLineNo">338</span>      } else {<a name="line.338"></a>
-<span class="sourceLineNo">339</span>        resetReader();<a 
name="line.339"></a>
-<span class="sourceLineNo">340</span>      }<a name="line.340"></a>
-<span class="sourceLineNo">341</span>    } catch (FileNotFoundException fnfe) 
{<a name="line.341"></a>
-<span class="sourceLineNo">342</span>      handleFileNotFound(path, fnfe);<a 
name="line.342"></a>
-<span class="sourceLineNo">343</span>    }  catch (RemoteException re) {<a 
name="line.343"></a>
-<span class="sourceLineNo">344</span>      IOException ioe = 
re.unwrapRemoteException(FileNotFoundException.class);<a name="line.344"></a>
-<span class="sourceLineNo">345</span>      if (!(ioe instanceof 
FileNotFoundException)) throw ioe;<a name="line.345"></a>
-<span class="sourceLineNo">346</span>      handleFileNotFound(path, 
(FileNotFoundException)ioe);<a name="line.346"></a>
-<span class="sourceLineNo">347</span>    } catch (LeaseNotRecoveredException 
lnre) {<a name="line.347"></a>
-<span class="sourceLineNo">348</span>      // HBASE-15019 the WAL was not 
closed due to some hiccup.<a name="line.348"></a>
-<span class="sourceLineNo">349</span>      LOG.warn("Try to recover the WAL 
lease " + currentPath, lnre);<a name="line.349"></a>
-<span class="sourceLineNo">350</span>      recoverLease(conf, currentPath);<a 
name="line.350"></a>
-<span class="sourceLineNo">351</span>      reader = null;<a 
name="line.351"></a>
-<span class="sourceLineNo">352</span>    } catch (NullPointerException npe) 
{<a name="line.352"></a>
-<span class="sourceLineNo">353</span>      // Workaround for race condition in 
HDFS-4380<a name="line.353"></a>
-<span class="sourceLineNo">354</span>      // which throws a NPE if we open a 
file before any data node has the most recent block<a name="line.354"></a>
-<span class="sourceLineNo">355</span>      // Just sleep and retry. Will 
require re-reading compressed WALs for compressionContext.<a 
name="line.355"></a>
-<span class="sourceLineNo">356</span>      LOG.warn("Got NPE opening reader, 
will retry.");<a name="line.356"></a>
-<span class="sourceLineNo">357</span>      reader = null;<a 
name="line.357"></a>
-<span class="sourceLineNo">358</span>    }<a name="line.358"></a>
-<span class="sourceLineNo">359</span>  }<a name="line.359"></a>
-<span class="sourceLineNo">360</span><a name="line.360"></a>
-<span class="sourceLineNo">361</span>  // For HBASE-15019<a 
name="line.361"></a>
-<span class="sourceLineNo">362</span>  private void recoverLease(final 
Configuration conf, final Path path) {<a name="line.362"></a>
-<span class="sourceLineNo">363</span>    try {<a name="line.363"></a>
-<span class="sourceLineNo">364</span>      final FileSystem dfs = 
FSUtils.getCurrentFileSystem(conf);<a name="line.364"></a>
-<span class="sourceLineNo">365</span>      FSUtils fsUtils = 
FSUtils.getInstance(dfs, conf);<a name="line.365"></a>
-<span class="sourceLineNo">366</span>      fsUtils.recoverFileLease(dfs, path, 
conf, new CancelableProgressable() {<a name="line.366"></a>
-<span class="sourceLineNo">367</span>        @Override<a name="line.367"></a>
-<span class="sourceLineNo">368</span>        public boolean progress() {<a 
name="line.368"></a>
-<span class="sourceLineNo">369</span>          LOG.debug("recover WAL lease: " 
+ path);<a name="line.369"></a>
-<span class="sourceLineNo">370</span>          return true;<a 
name="line.370"></a>
-<span class="sourceLineNo">371</span>        }<a name="line.371"></a>
-<span class="sourceLineNo">372</span>      });<a name="line.372"></a>
-<span class="sourceLineNo">373</span>    } catch (IOException e) {<a 
name="line.373"></a>
-<span class="sourceLineNo">374</span>      LOG.warn("unable to recover lease 
for WAL: " + path, e);<a name="line.374"></a>
-<span class="sourceLineNo">375</span>    }<a name="line.375"></a>
-<span class="sourceLineNo">376</span>  }<a name="line.376"></a>
-<span class="sourceLineNo">377</span><a name="line.377"></a>
-<span class="sourceLineNo">378</span>  private void resetReader() throws 
IOException {<a name="line.378"></a>
-<span class="sourceLineNo">379</span>    try {<a name="line.379"></a>
-<span class="sourceLineNo">380</span>      reader.reset();<a 
name="line.380"></a>
-<span class="sourceLineNo">381</span>      seek();<a name="line.381"></a>
-<span class="sourceLineNo">382</span>    } catch (FileNotFoundException fnfe) 
{<a name="line.382"></a>
-<span class="sourceLineNo">383</span>      // If the log was archived, 
continue reading from there<a name="line.383"></a>
-<span class="sourceLineNo">384</span>      Path archivedLog = 
getArchivedLog(currentPath);<a name="line.384"></a>
-<span class="sourceLineNo">385</span>      if 
(!currentPath.equals(archivedLog)) {<a name="line.385"></a>
-<span class="sourceLineNo">386</span>        openReader(archivedLog);<a 
name="line.386"></a>
-<span class="sourceLineNo">387</span>      } else {<a name="line.387"></a>
-<span class="sourceLineNo">388</span>        throw fnfe;<a name="line.388"></a>
-<span class="sourceLineNo">389</span>      }<a name="line.389"></a>
-<span class="sourceLineNo">390</span>    } catch (NullPointerException npe) 
{<a name="line.390"></a>
-<span class="sourceLineNo">391</span>      throw new IOException("NPE 
resetting reader, likely HDFS-4380", npe);<a name="line.391"></a>
-<span class="sourceLineNo">392</span>    }<a name="line.392"></a>
-<span class="sourceLineNo">393</span>  }<a name="line.393"></a>
-<span class="sourceLineNo">394</span><a name="line.394"></a>
-<span class="sourceLineNo">395</span>  private void seek() throws IOException 
{<a name="line.395"></a>
-<span class="sourceLineNo">396</span>    if (currentPosition != 0) {<a 
name="line.396"></a>
-<span class="sourceLineNo">397</span>      reader.seek(currentPosition);<a 
name="line.397"></a>
-<span class="sourceLineNo">398</span>    }<a name="line.398"></a>
-<span class="sourceLineNo">399</span>  }<a name="line.399"></a>
-<span class="sourceLineNo">400</span><a name="line.400"></a>
-<span class="sourceLineNo">401</span>  private long currentTrailerSize() {<a 
name="line.401"></a>
-<span class="sourceLineNo">402</span>    long size = -1L;<a 
name="line.402"></a>
-<span class="sourceLineNo">403</span>    if (reader instanceof 
ProtobufLogReader) {<a name="line.403"></a>
-<span class="sourceLineNo">404</span>      final ProtobufLogReader pblr = 
(ProtobufLogReader) reader;<a name="line.404"></a>
-<span class="sourceLineNo">405</span>      size = pblr.trailerSize();<a 
name="line.405"></a>
-<span class="sourceLineNo">406</span>    }<a name="line.406"></a>
-<span class="sourceLineNo">407</span>    return size;<a name="line.407"></a>
-<span class="sourceLineNo">408</span>  }<a name="line.408"></a>
-<span class="sourceLineNo">409</span><a name="line.409"></a>
-<span class="sourceLineNo">410</span>  @InterfaceAudience.Private<a 
name="line.410"></a>
-<span class="sourceLineNo">411</span>  public static class 
WALEntryStreamRuntimeException extends RuntimeException {<a name="line.411"></a>
-<span class="sourceLineNo">412</span>    private static final long 
serialVersionUID = -6298201811259982568L;<a name="line.412"></a>
-<span class="sourceLineNo">413</span><a name="line.413"></a>
-<span class="sourceLineNo">414</span>    public 
WALEntryStreamRuntimeException(Exception e) {<a name="line.414"></a>
-<span class="sourceLineNo">415</span>      super(e);<a name="line.415"></a>
-<span class="sourceLineNo">416</span>    }<a name="line.416"></a>
-<span class="sourceLineNo">417</span>  }<a name="line.417"></a>
-<span class="sourceLineNo">418</span><a name="line.418"></a>
-<span class="sourceLineNo">419</span>}<a name="line.419"></a>
+<span class="sourceLineNo">236</span>          if (LOG.isDebugEnabled()) {<a 
name="line.236"></a>
+<span class="sourceLineNo">237</span>            LOG.debug("Reached the end of 
WAL file '" + currentPath<a name="line.237"></a>
+<span class="sourceLineNo">238</span>                + "'. It was not closed 
cleanly, so we did not parse " + skippedBytes<a name="line.238"></a>
+<span class="sourceLineNo">239</span>                + " bytes of data. This 
is normally ok.");<a name="line.239"></a>
+<span class="sourceLineNo">240</span>          }<a name="line.240"></a>
+<span class="sourceLineNo">241</span>          
metrics.incrUncleanlyClosedWALs();<a name="line.241"></a>
+<span class="sourceLineNo">242</span>          
metrics.incrBytesSkippedInUncleanlyClosedWALs(skippedBytes);<a 
name="line.242"></a>
+<span class="sourceLineNo">243</span>        }<a name="line.243"></a>
+<span class="sourceLineNo">244</span>      } else if (currentPosition + 
trailerSize &lt; stat.getLen()) {<a name="line.244"></a>
+<span class="sourceLineNo">245</span>        LOG.warn("Processing end of WAL 
file '" + currentPath + "'. At position " + currentPosition<a 
name="line.245"></a>
+<span class="sourceLineNo">246</span>            + ", which is too far away 
from reported file length " + stat.getLen()<a name="line.246"></a>
+<span class="sourceLineNo">247</span>            + ". Restarting WAL reading 
(see HBASE-15983 for details). " + getCurrentPathStat());<a name="line.247"></a>
+<span class="sourceLineNo">248</span>        setPosition(0);<a 
name="line.248"></a>
+<span class="sourceLineNo">249</span>        resetReader();<a 
name="line.249"></a>
+<span class="sourceLineNo">250</span>        
metrics.incrRestartedWALReading();<a name="line.250"></a>
+<span class="sourceLineNo">251</span>        
metrics.incrRepeatedFileBytes(currentPosition);<a name="line.251"></a>
+<span class="sourceLineNo">252</span>        return false;<a 
name="line.252"></a>
+<span class="sourceLineNo">253</span>      }<a name="line.253"></a>
+<span class="sourceLineNo">254</span>    }<a name="line.254"></a>
+<span class="sourceLineNo">255</span>    if (LOG.isTraceEnabled()) {<a 
name="line.255"></a>
+<span class="sourceLineNo">256</span>      LOG.trace("Reached the end of log " 
+ this.currentPath + ", and the length of the file is "<a name="line.256"></a>
+<span class="sourceLineNo">257</span>          + (stat == null ? "N/A" : 
stat.getLen()));<a name="line.257"></a>
+<span class="sourceLineNo">258</span>    }<a name="line.258"></a>
+<span class="sourceLineNo">259</span>    metrics.incrCompletedWAL();<a 
name="line.259"></a>
+<span class="sourceLineNo">260</span>    return true;<a name="line.260"></a>
+<span class="sourceLineNo">261</span>  }<a name="line.261"></a>
+<span class="sourceLineNo">262</span><a name="line.262"></a>
+<span class="sourceLineNo">263</span>  private void dequeueCurrentLog() throws 
IOException {<a name="line.263"></a>
+<span class="sourceLineNo">264</span>    if (LOG.isDebugEnabled()) {<a 
name="line.264"></a>
+<span class="sourceLineNo">265</span>      LOG.debug("Reached the end of log " 
+ currentPath);<a name="line.265"></a>
+<span class="sourceLineNo">266</span>    }<a name="line.266"></a>
+<span class="sourceLineNo">267</span>    closeReader();<a name="line.267"></a>
+<span class="sourceLineNo">268</span>    logQueue.remove();<a 
name="line.268"></a>
+<span class="sourceLineNo">269</span>    setPosition(0);<a name="line.269"></a>
+<span class="sourceLineNo">270</span>    metrics.decrSizeOfLogQueue();<a 
name="line.270"></a>
+<span class="sourceLineNo">271</span>  }<a name="line.271"></a>
+<span class="sourceLineNo">272</span><a name="line.272"></a>
+<span class="sourceLineNo">273</span>  private void 
readNextEntryAndSetPosition() throws IOException {<a name="line.273"></a>
+<span class="sourceLineNo">274</span>    Entry readEntry = reader.next();<a 
name="line.274"></a>
+<span class="sourceLineNo">275</span>    long readerPos = 
reader.getPosition();<a name="line.275"></a>
+<span class="sourceLineNo">276</span>    if (readEntry != null) {<a 
name="line.276"></a>
+<span class="sourceLineNo">277</span>      metrics.incrLogEditsRead();<a 
name="line.277"></a>
+<span class="sourceLineNo">278</span>      
metrics.incrLogReadInBytes(readerPos - currentPosition);<a name="line.278"></a>
+<span class="sourceLineNo">279</span>    }<a name="line.279"></a>
+<span class="sourceLineNo">280</span>    currentEntry = readEntry; // could be 
null<a name="line.280"></a>
+<span class="sourceLineNo">281</span>    setPosition(readerPos);<a 
name="line.281"></a>
+<span class="sourceLineNo">282</span>  }<a name="line.282"></a>
+<span class="sourceLineNo">283</span><a name="line.283"></a>
+<span class="sourceLineNo">284</span>  private void closeReader() throws 
IOException {<a name="line.284"></a>
+<span class="sourceLineNo">285</span>    if (reader != null) {<a 
name="line.285"></a>
+<span class="sourceLineNo">286</span>      reader.close();<a 
name="line.286"></a>
+<span class="sourceLineNo">287</span>      reader = null;<a 
name="line.287"></a>
+<span class="sourceLineNo">288</span>    }<a name="line.288"></a>
+<span class="sourceLineNo">289</span>  }<a name="line.289"></a>
+<span class="sourceLineNo">290</span><a name="line.290"></a>
+<span class="sourceLineNo">291</span>  // if we don't have a reader, open a 
reader on the next log<a name="line.291"></a>
+<span class="sourceLineNo">292</span>  private boolean checkReader() throws 
IOException {<a name="line.292"></a>
+<span class="sourceLineNo">293</span>    if (reader == null) {<a 
name="line.293"></a>
+<span class="sourceLineNo">294</span>      return openNextLog();<a 
name="line.294"></a>
+<span class="sourceLineNo">295</span>    }<a name="line.295"></a>
+<span class="sourceLineNo">296</span>    return true;<a name="line.296"></a>
+<span class="sourceLineNo">297</span>  }<a name="line.297"></a>
+<span class="sourceLineNo">298</span><a name="line.298"></a>
+<span class="sourceLineNo">299</span>  // open a reader on the next log in 
queue<a name="line.299"></a>
+<span class="sourceLineNo">300</span>  private boolean openNextLog() throws 
IOException {<a name="line.300"></a>
+<span class="sourceLineNo">301</span>    Path nextPath = logQueue.peek();<a 
name="line.301"></a>
+<span class="sourceLineNo">302</span>    if (nextPath != null) {<a 
name="line.302"></a>
+<span class="sourceLineNo">303</span>      openReader(nextPath);<a 
name="line.303"></a>
+<span class="sourceLineNo">304</span>      if (reader != null) return true;<a 
name="line.304"></a>
+<span class="sourceLineNo">305</span>    }<a name="line.305"></a>
+<span class="sourceLineNo">306</span>    return false;<a name="line.306"></a>
+<span class="sourceLineNo">307</span>  }<a name="line.307"></a>
+<span class="sourceLineNo">308</span><a name="line.308"></a>
+<span class="sourceLineNo">309</span>  private Path getArchivedLog(Path path) 
throws IOException {<a name="line.309"></a>
+<span class="sourceLineNo">310</span>    Path rootDir = 
FSUtils.getRootDir(conf);<a name="line.310"></a>
+<span class="sourceLineNo">311</span>    Path oldLogDir = new Path(rootDir, 
HConstants.HREGION_OLDLOGDIR_NAME);<a name="line.311"></a>
+<span class="sourceLineNo">312</span>    Path archivedLogLocation = new 
Path(oldLogDir, path.getName());<a name="line.312"></a>
+<span class="sourceLineNo">313</span>    if (fs.exists(archivedLogLocation)) 
{<a name="line.313"></a>
+<span class="sourceLineNo">314</span>      LOG.info("Log " + path + " was 
moved to " + archivedLogLocation);<a name="line.314"></a>
+<span class="sourceLineNo">315</span>      return archivedLogLocation;<a 
name="line.315"></a>
+<span class="sourceLineNo">316</span>    } else {<a name="line.316"></a>
+<span class="sourceLineNo">317</span>      LOG.error("Couldn't locate log: " + 
path);<a name="line.317"></a>
+<span class="sourceLineNo">318</span>      return path;<a name="line.318"></a>
+<span class="sourceLineNo">319</span>    }<a name="line.319"></a>
+<span class="sourceLineNo">320</span>  }<a name="line.320"></a>
+<span class="sourceLineNo">321</span><a name="line.321"></a>
+<span class="sourceLineNo">322</span>  private void handleFileNotFound(Path 
path, FileNotFoundException fnfe) throws IOException {<a name="line.322"></a>
+<span class="sourceLineNo">323</span>    // If the log was archived, continue 
reading from there<a name="line.323"></a>
+<span class="sourceLineNo">324</span>    Path archivedLog = 
getArchivedLog(path);<a name="line.324"></a>
+<span class="sourceLineNo">325</span>    if (!path.equals(archivedLog)) {<a 
name="line.325"></a>
+<span class="sourceLineNo">326</span>      openReader(archivedLog);<a 
name="line.326"></a>
+<span class="sourceLineNo">327</span>    } else {<a name="line.327"></a>
+<span class="sourceLineNo">328</span>      throw fnfe;<a name="line.328"></a>
+<span class="sourceLineNo">329</span>    }<a name="line.329"></a>
+<span class="sourceLineNo">330</span>  }<a name="line.330"></a>
+<span class="sourceLineNo">331</span>  private void openReader(Path path) 
throws IOException {<a name="line.331"></a>
+<span class="sourceLineNo">332</span>    try {<a name="line.332"></a>
+<span class="sourceLineNo">333</span>      // Detect if this is a new file, if 
so get a new reader else<a name="line.333"></a>
+<span class="sourceLineNo">334</span>      // reset the current reader so that 
we see the new data<a name="line.334"></a>
+<span class="sourceLineNo">335</span>      if (reader == null || 
!getCurrentPath().equals(path)) {<a name="line.335"></a>
+<span class="sourceLineNo">336</span>        closeReader();<a 
name="line.336"></a>
+<span class="sourceLineNo">337</span>        reader = 
WALFactory.createReader(fs, path, conf);<a name="line.337"></a>
+<span class="sourceLineNo">338</span>        seek();<a name="line.338"></a>
+<span class="sourceLineNo">339</span>        setCurrentPath(path);<a 
name="line.339"></a>
+<span class="sourceLineNo">340</span>      } else {<a name="line.340"></a>
+<span class="sourceLineNo">341</span>        resetReader();<a 
name="line.341"></a>
+<span class="sourceLineNo">342</span>      }<a name="line.342"></a>
+<span class="sourceLineNo">343</span>    } catch (FileNotFoundException fnfe) 
{<a name="line.343"></a>
+<span class="sourceLineNo">344</span>      handleFileNotFound(path, fnfe);<a 
name="line.344"></a>
+<span class="sourceLineNo">345</span>    }  catch (RemoteException re) {<a 
name="line.345"></a>
+<span class="sourceLineNo">346</span>      IOException ioe = 
re.unwrapRemoteException(FileNotFoundException.class);<a name="line.346"></a>
+<span class="sourceLineNo">347</span>      if (!(ioe instanceof 
FileNotFoundException)) throw ioe;<a name="line.347"></a>
+<span class="sourceLineNo">348</span>      handleFileNotFound(path, 
(FileNotFoundException)ioe);<a name="line.348"></a>
+<span class="sourceLineNo">349</span>    } catch (LeaseNotRecoveredException 
lnre) {<a name="line.349"></a>
+<span class="sourceLineNo">350</span>      // HBASE-15019 the WAL was not 
closed due to some hiccup.<a name="line.350"></a>
+<span class="sourceLineNo">351</span>      LOG.warn("Try to recover the WAL 
lease " + currentPath, lnre);<a name="line.351"></a>
+<span class="sourceLineNo">352</span>      recoverLease(conf, currentPath);<a 
name="line.352"></a>
+<span class="sourceLineNo">353</span>      reader = null;<a 
name="line.353"></a>
+<span class="sourceLineNo">354</span>    } catch (NullPointerException npe) 
{<a name="line.354"></a>
+<span class="sourceLineNo">355</span>      // Workaround for race condition in 
HDFS-4380<a name="line.355"></a>
+<span class="sourceLineNo">356</span>      // which throws a NPE if we open a 
file before any data node has the most recent block<a name="line.356"></a>
+<span class="sourceLineNo">357</span>      // Just sleep and retry. Will 
require re-reading compressed WALs for compressionContext.<a 
name="line.357"></a>
+<span class="sourceLineNo">358</span>      LOG.warn("Got NPE opening reader, 
will retry.");<a name="line.358"></a>
+<span class="sourceLineNo">359</span>      reader = null;<a 
name="line.359"></a>
+<span class="sourceLineNo">360</span>    }<a name="line.360"></a>
+<span class="sourceLineNo">361</span>  }<a name="line.361"></a>
+<span class="sourceLineNo">362</span><a name="line.362"></a>
+<span class="sourceLineNo">363</span>  // For HBASE-15019<a 
name="line.363"></a>
+<span class="sourceLineNo">364</span>  private void recoverLease(final 
Configuration conf, final Path path) {<a name="line.364"></a>
+<span class="sourceLineNo">365</span>    try {<a name="line.365"></a>
+<span class="sourceLineNo">366</span>      final FileSystem dfs = 
FSUtils.getCurrentFileSystem(conf);<a name="line.366"></a>
+<span class="sourceLineNo">367</span>      FSUtils fsUtils = 
FSUtils.getInstance(dfs, conf);<a name="line.367"></a>
+<span class="sourceLineNo">368</span>      fsUtils.recoverFileLease(dfs, path, 
conf, new CancelableProgressable() {<a name="line.368"></a>
+<span class="sourceLineNo">369</span>        @Override<a name="line.369"></a>
+<span class="sourceLineNo">370</span>        public boolean progress() {<a 
name="line.370"></a>
+<span class="sourceLineNo">371</span>          LOG.debug("recover WAL lease: " 
+ path);<a name="line.371"></a>
+<span class="sourceLineNo">372</span>          return true;<a 
name="line.372"></a>
+<span class="sourceLineNo">373</span>        }<a name="line.373"></a>
+<span class="sourceLineNo">374</span>      });<a name="line.374"></a>
+<span class="sourceLineNo">375</span>    } catch (IOException e) {<a 
name="line.375"></a>
+<span class="sourceLineNo">376</span>      LOG.warn("unable to recover lease 
for WAL: " + path, e);<a name="line.376"></a>
+<span class="sourceLineNo">377</span>    }<a name="line.377"></a>
+<span class="sourceLineNo">378</span>  }<a name="line.378"></a>
+<span class="sourceLineNo">379</span><a name="line.379"></a>
+<span class="sourceLineNo">380</span>  private void resetReader() throws 
IOException {<a name="line.380"></a>
+<span class="sourceLineNo">381</span>    try {<a name="line.381"></a>
+<span class="sourceLineNo">382</span>      reader.reset();<a 
name="line.382"></a>
+<span class="sourceLineNo">383</span>      seek();<a name="line.383"></a>
+<span class="sourceLineNo">384</span>    } catch (FileNotFoundException fnfe) 
{<a name="line.384"></a>
+<span class="sourceLineNo">385</span>      // If the log was archived, 
continue reading from there<a name="line.385"></a>
+<span class="sourceLineNo">386</span>      Path archivedLog = 
getArchivedLog(currentPath);<a name="line.386"></a>
+<span class="sourceLineNo">387</span>      if 
(!currentPath.equals(archivedLog)) {<a name="line.387"></a>
+<span class="sourceLineNo">388</span>        openReader(archivedLog);<a 
name="line.388"></a>
+<span class="sourceLineNo">389</span>      } else {<a name="line.389"></a>
+<span class="sourceLineNo">390</span>        throw fnfe;<a name="line.390"></a>
+<span class="sourceLineNo">391</span>      }<a name="line.391"></a>
+<span class="sourceLineNo">392</span>    } catch (NullPointerException npe) 
{<a name="line.392"></a>
+<span class="sourceLineNo">393</span>      throw new IOException("NPE 
resetting reader, likely HDFS-4380", npe);<a name="line.393"></a>
+<span class="sourceLineNo">394</span>    }<a name="line.394"></a>
+<span class="sourceLineNo">395</span>  }<a name="line.395"></a>
+<span class="sourceLineNo">396</span><a name="line.396"></a>
+<span class="sourceLineNo">397</span>  private void seek() throws IOException 
{<a name="line.397"></a>
+<span class="sourceLineNo">398</span>    if (currentPosition != 0) {<a 
name="line.398"></a>
+<span class="sourceLineNo">399</span>      reader.seek(currentPosition);<a 
name="line.399"></a>
+<span class="sourceLineNo">400</span>    }<a name="line.400"></a>
+<span class="sourceLineNo">401</span>  }<a name="line.401"></a>
+<span class="sourceLineNo">402</span><a name="line.402"></a>
+<span class="sourceLineNo">403</span>  private long currentTrailerSize() {<a 
name="line.403"></a>
+<span class="sourceLineNo">404</span>    long size = -1L;<a 
name="line.404"></a>
+<span class="sourceLineNo">405</span>    if (reader instanceof 
ProtobufLogReader) {<a name="line.405"></a>
+<span class="sourceLineNo">406</span>      final ProtobufLogReader pblr = 
(ProtobufLogReader) reader;<a name="line.406"></a>
+<span class="sourceLineNo">407</span>      size = pblr.trailerSize();<a 
name="line.407"></a>
+<span class="sourceLineNo">408</span>    }<a name="line.408"></a>
+<span class="sourceLineNo">409</span>    return size;<a name="line.409"></a>
+<span class="sourceLineNo">410</span>  }<a name="line.410"></a>
+<span class="sourceLineNo">411</span><a name="line.411"></a>
+<span class="sourceLineNo">412</span>  @InterfaceAudience.Private<a 
name="line.412"></a>
+<span class="sourceLineNo">413</span>  public static class 
WALEntryStreamRuntimeException extends RuntimeException {<a name="line.413"></a>
+<span class="sourceLineNo">414</span>    private static final long 
serialVersionUID = -6298201811259982568L;<a name="line.414"></a>
+<span class="sourceLineNo">415</span><a name="line.415"></a>
+<span class="sourceLineNo">416</span>    public 
WALEntryStreamRuntimeException(Exception e) {<a name="line.416"></a>
+<span class="sourceLineNo">417</span>      super(e);<a name="line.417"></a>
+<span class="sourceLineNo">418</span>    }<a name="line.418"></a>
+<span class="sourceLineNo">419</span>  }<a name="line.419"></a>
+<span class="sourceLineNo">420</span><a name="line.420"></a>
+<span class="sourceLineNo">421</span>}<a name="line.421"></a>
 
 
 

Reply via email to