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 2019/10/12 16:16:47 UTC
[hbase] branch branch-2.2 updated: HBASE-23154 list_deadservers
return incorrect no of rows (#717)
This is an automated email from the ASF dual-hosted git repository.
stack pushed a commit to branch branch-2.2
in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/branch-2.2 by this push:
new 71703af HBASE-23154 list_deadservers return incorrect no of rows (#717)
71703af is described below
commit 71703af4ac1cf79c2cb2fd103909c1dc366a33d2
Author: Karthik Palanisamy <kp...@hortonworks.com>
AuthorDate: Sat Oct 12 09:13:50 2019 -0700
HBASE-23154 list_deadservers return incorrect no of rows (#717)
Signed-off-by: stack <st...@apache.org>
---
hbase-shell/src/main/ruby/shell/commands/list_deadservers.rb | 2 +-
hbase-shell/src/test/ruby/hbase/admin_test.rb | 5 +++++
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/hbase-shell/src/main/ruby/shell/commands/list_deadservers.rb b/hbase-shell/src/main/ruby/shell/commands/list_deadservers.rb
index 69c7e72..735fdaa 100644
--- a/hbase-shell/src/main/ruby/shell/commands/list_deadservers.rb
+++ b/hbase-shell/src/main/ruby/shell/commands/list_deadservers.rb
@@ -37,7 +37,7 @@ module Shell
formatter.row([server.toString])
end
- formatter.footer(now, servers.size)
+ formatter.footer(servers.size)
end
end
end
diff --git a/hbase-shell/src/test/ruby/hbase/admin_test.rb b/hbase-shell/src/test/ruby/hbase/admin_test.rb
index 5cd3024..b6c49a5 100644
--- a/hbase-shell/src/test/ruby/hbase/admin_test.rb
+++ b/hbase-shell/src/test/ruby/hbase/admin_test.rb
@@ -98,6 +98,11 @@ module Hbase
assert(list.count > 0)
end
+ define_test 'list_deadservers should return exact count of dead servers' do
+ output = capture_stdout { command(:list_deadservers) }
+ assert(output.include?('0 row(s)'))
+ end
+
#-------------------------------------------------------------------------------
define_test "flush should work" do