You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ns...@apache.org on 2011/10/11 21:13:12 UTC

svn commit: r1182035 - /hbase/branches/0.89/src/main/ruby/hbase/admin.rb

Author: nspiegelberg
Date: Tue Oct 11 19:13:12 2011
New Revision: 1182035

URL: http://svn.apache.org/viewvc?rev=1182035&view=rev
Log:
alter_status shell prints sensible message at completion

Summary:
The alter_status command used to print 0/0 once an alter operation had
completed and its progress was no longer available. Now it instad
indicates that all regions were updated.

Test Plan: Tested with alter shell command on a dev cluster.

Reviewers: kannan

Reviewed By: kannan

CC: hbase-eng@lists, kannan

Differential Revision: 336201

Task ID: 693561

Modified:
    hbase/branches/0.89/src/main/ruby/hbase/admin.rb

Modified: hbase/branches/0.89/src/main/ruby/hbase/admin.rb
URL: http://svn.apache.org/viewvc/hbase/branches/0.89/src/main/ruby/hbase/admin.rb?rev=1182035&r1=1182034&r2=1182035&view=diff
==============================================================================
--- hbase/branches/0.89/src/main/ruby/hbase/admin.rb (original)
+++ hbase/branches/0.89/src/main/ruby/hbase/admin.rb Tue Oct 11 19:13:12 2011
@@ -227,7 +227,11 @@ module Hbase
          status = Pair.new()
          begin
            status = @admin.getAlterStatus(table_name.to_java_bytes)
-           puts "#{status.getSecond() - status.getFirst()}/#{status.getSecond()} regions updated."
+           if status.getSecond() != 0
+             puts "#{status.getSecond() - status.getFirst()}/#{status.getSecond()} regions updated."
+           else
+             puts "All regions updated."
+           end
            sleep 1
          end while status != nil && status.getFirst() != 0
          puts "Done."