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 2009/08/04 01:21:47 UTC

svn commit: r800619 - in /hadoop/hbase/branches/0.20: CHANGES.txt src/java/org/apache/hadoop/hbase/regionserver/HLog.java

Author: stack
Date: Mon Aug  3 23:21:47 2009
New Revision: 800619

URL: http://svn.apache.org/viewvc?rev=800619&view=rev
Log:
HBASE-1739 hbase-1683 broke splitting; only split three logs no matter what N was

Modified:
    hadoop/hbase/branches/0.20/CHANGES.txt
    hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HLog.java

Modified: hadoop/hbase/branches/0.20/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.20/CHANGES.txt?rev=800619&r1=800618&r2=800619&view=diff
==============================================================================
--- hadoop/hbase/branches/0.20/CHANGES.txt (original)
+++ hadoop/hbase/branches/0.20/CHANGES.txt Mon Aug  3 23:21:47 2009
@@ -311,6 +311,8 @@
    HBASE-1732  Flag to disable regionserver restart
    HBASE-1604  HBaseClient.getConnection() may return a broken connection
                without throwing an exception (Eugene Kirpichov via Stack)
+   HBASE-1739  hbase-1683 broke splitting; only split three logs no matter
+               what N was
 
   IMPROVEMENTS
    HBASE-1089  Add count of regions on filesystem to master UI; add percentage

Modified: hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HLog.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HLog.java?rev=800619&r1=800618&r2=800619&view=diff
==============================================================================
--- hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HLog.java (original)
+++ hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HLog.java Mon Aug  3 23:21:47 2009
@@ -860,7 +860,7 @@
         // Stop at logfiles.length when it's the last step
         int endIndex = step == maxSteps - 1? logfiles.length: 
           step * concurrentLogReads + concurrentLogReads;
-        for (int i = (step * 10); i < endIndex; i++) {
+        for (int i = (step * concurrentLogReads); i < endIndex; i++) {
           // Check for possibly empty file. With appends, currently Hadoop 
           // reports a zero length even if the file has been sync'd. Revisit if
           // HADOOP-4751 is committed.