You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by jd...@apache.org on 2009/12/09 00:55:36 UTC

svn commit: r888640 - in /hadoop/hbase/trunk: CHANGES.txt bin/HBase.rb

Author: jdcryans
Date: Tue Dec  8 23:55:36 2009
New Revision: 888640

URL: http://svn.apache.org/viewvc?rev=888640&view=rev
Log:
HBASE-2033  Shell scan 'limit' is off by one

Modified:
    hadoop/hbase/trunk/CHANGES.txt
    hadoop/hbase/trunk/bin/HBase.rb

Modified: hadoop/hbase/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/CHANGES.txt?rev=888640&r1=888639&r2=888640&view=diff
==============================================================================
--- hadoop/hbase/trunk/CHANGES.txt (original)
+++ hadoop/hbase/trunk/CHANGES.txt Tue Dec  8 23:55:36 2009
@@ -122,6 +122,7 @@
    HBASE-1994  Master will lose hlog entries while splitting if region has
                empty oldlogfile.log (Lars George via Stack)
    HBASE-2022  NPE in housekeeping kills RS
+   HBASE-2033  Shell scan 'limit' is off by one
 
   IMPROVEMENTS
    HBASE-1760  Cleanup TODOs in HTable

Modified: hadoop/hbase/trunk/bin/HBase.rb
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/bin/HBase.rb?rev=888640&r1=888639&r2=888640&view=diff
==============================================================================
--- hadoop/hbase/trunk/bin/HBase.rb (original)
+++ hadoop/hbase/trunk/bin/HBase.rb Tue Dec  8 23:55:36 2009
@@ -426,7 +426,6 @@
       while i.hasNext()
         r = i.next()
         row = String.from_java_bytes r.getRow()
-        count += 1
         if limit != -1 and count >= limit
           break
         end
@@ -437,6 +436,7 @@
           cell = toString(column, kv, maxlength)
           @formatter.row([row, "column=%s, %s" % [column, cell]])
         end
+        count += 1
       end
       @formatter.footer(now, count)
     end