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 2008/08/01 23:23:03 UTC

svn commit: r681833 - /hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HStore.java

Author: stack
Date: Fri Aug  1 14:23:02 2008
New Revision: 681833

URL: http://svn.apache.org/viewvc?rev=681833&view=rev
Log:
Fix overlogging; part of HBASE-792 patch

Modified:
    hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HStore.java

Modified: hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HStore.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HStore.java?rev=681833&r1=681832&r2=681833&view=diff
==============================================================================
--- hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HStore.java (original)
+++ hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HStore.java Fri Aug  1 14:23:02 2008
@@ -1524,7 +1524,7 @@
   private static boolean isExpired(final HStoreKey hsk, final long ttl,
       final long now) {
     boolean result = ttl != HConstants.FOREVER && now > hsk.getTimestamp() + ttl;
-    if (LOG.isDebugEnabled()) {
+    if (result && LOG.isDebugEnabled()) {
       LOG.debug("rowAtOrBeforeCandidate 1:" + hsk +
         ": expired, skipped");
     }
@@ -1887,4 +1887,4 @@
       return key;
     }
   }
-}
\ No newline at end of file
+}