You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by je...@apache.org on 2013/10/04 20:42:57 UTC

svn commit: r1529252 - /hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java

Author: jeffreyz
Date: Fri Oct  4 18:42:57 2013
New Revision: 1529252

URL: http://svn.apache.org/r1529252
Log:
HBASE-9709: LogReplay throws NPE when no KVs to be replayed in a WALEdit

Modified:
    hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java

Modified: hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java?rev=1529252&r1=1529251&r2=1529252&view=diff
==============================================================================
--- hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java (original)
+++ hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java Fri Oct  4 18:42:57 2013
@@ -1478,7 +1478,7 @@ public class HLogSplitter {
         }
 
         // skip the edit
-        if (needSkip) continue;
+        if (loc == null || needSkip) continue;
 
         if (!skippedKVs.isEmpty()) {
           kvs.removeAll(skippedKVs);