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:56:51 UTC

svn commit: r888642 - in /hadoop/hbase/branches/0.20: CHANGES.txt bin/HBase.rb

Author: jdcryans
Date: Tue Dec  8 23:56:51 2009
New Revision: 888642

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

Modified:
    hadoop/hbase/branches/0.20/CHANGES.txt
    hadoop/hbase/branches/0.20/bin/HBase.rb

Modified: hadoop/hbase/branches/0.20/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.20/CHANGES.txt?rev=888642&r1=888641&r2=888642&view=diff
==============================================================================
--- hadoop/hbase/branches/0.20/CHANGES.txt (original)
+++ hadoop/hbase/branches/0.20/CHANGES.txt Tue Dec  8 23:56:51 2009
@@ -12,6 +12,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-1961  HBase EC2 scripts

Modified: hadoop/hbase/branches/0.20/bin/HBase.rb
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.20/bin/HBase.rb?rev=888642&r1=888641&r2=888642&view=diff
==============================================================================
--- hadoop/hbase/branches/0.20/bin/HBase.rb (original)
+++ hadoop/hbase/branches/0.20/bin/HBase.rb Tue Dec  8 23:56:51 2009
@@ -440,7 +440,6 @@
       while i.hasNext()
         r = i.next().getRowResult()
         row = String.from_java_bytes r.getRow()
-        count += 1
         if limit != -1 and count >= limit
           break
         end
@@ -449,6 +448,7 @@
           cell = toString(column, v, maxlength)
           @formatter.row([row, "column=%s, %s" % [column, cell]])
         end
+        count += 1
       end
       @formatter.footer(now, count)
     end