You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2011/03/22 03:05:06 UTC

svn commit: r1084052 - in /hbase/branches/0.90: CHANGES.txt src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java

Author: stack
Date: Tue Mar 22 02:05:05 2011
New Revision: 1084052

URL: http://svn.apache.org/viewvc?rev=1084052&view=rev
Log:
HBASE-3674 Treat ChecksumException as we would a ParseException splitting logs; else we replay split on every restart

Modified:
    hbase/branches/0.90/CHANGES.txt
    hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java

Modified: hbase/branches/0.90/CHANGES.txt
URL: http://svn.apache.org/viewvc/hbase/branches/0.90/CHANGES.txt?rev=1084052&r1=1084051&r2=1084052&view=diff
==============================================================================
--- hbase/branches/0.90/CHANGES.txt (original)
+++ hbase/branches/0.90/CHANGES.txt Tue Mar 22 02:05:05 2011
@@ -42,6 +42,8 @@ Release 0.90.2 - Unreleased
                till 0.92; Race between split and OPENED processing
    HBASE-3641  LruBlockCache.CacheStats.getHitCount() is not using the
                correct variable
+   HBASE-3674  Treat ChecksumException as we would a ParseException splitting
+               logs; else we replay split on every restart
 
   IMPROVEMENTS
    HBASE-3542  MultiGet methods in Thrift

Modified: hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java?rev=1084052&r1=1084051&r2=1084052&view=diff
==============================================================================
--- hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java (original)
+++ hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java Tue Mar 22 02:05:05 2011
@@ -243,7 +243,7 @@ public class HLogSplitter {
     List<Path> corruptedLogs = new ArrayList<Path>();
     List<Path> splits = null;
 
-    boolean skipErrors = conf.getBoolean("hbase.hlog.split.skip.errors", false);
+    boolean skipErrors = conf.getBoolean("hbase.hlog.split.skip.errors", true);
 
     splitSize = 0;
 
@@ -276,7 +276,7 @@ public class HLogSplitter {
           // If the IOE resulted from bad file format,
           // then this problem is idempotent and retrying won't help
           if (e.getCause() instanceof ParseException) {
-            LOG.warn("ParseException from hlog " + logPath + ".  continuing");
+            LOG.warn("Parse exception from hlog " + logPath + ".  continuing", e);
             processedLogs.add(logPath);
           } else {
             if (skipErrors) {