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 2013/08/30 18:13:09 UTC

svn commit: r1519011 - /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java

Author: stack
Date: Fri Aug 30 16:13:08 2013
New Revision: 1519011

URL: http://svn.apache.org/r1519011
Log:
HBASE-7954 Fix the retrying logic of memstore flushes to avoid extra sleep

Modified:
    hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java

Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java?rev=1519011&r1=1519010&r2=1519011&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java Fri Aug 30 16:13:08 2013
@@ -805,7 +805,7 @@ public class Store extends SchemaConfigu
         LOG.warn("Failed flushing store file, retring num=" + i, e);
         lastException = e;
       }
-      if (lastException != null) {
+      if (lastException != null && i < (flush_retries_number - 1)) {
         try {
           Thread.sleep(pauseTime);
         } catch (InterruptedException e) {