You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by vj...@apache.org on 2020/03/06 07:40:06 UTC

[hbase] branch branch-1 updated: HBASE-23909 : list_regions raise correct error if it is being splitted/merged/transitioning (#1238)

This is an automated email from the ASF dual-hosted git repository.

vjasani pushed a commit to branch branch-1
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-1 by this push:
     new 1931714  HBASE-23909 : list_regions raise correct error if it is being splitted/merged/transitioning (#1238)
1931714 is described below

commit 1931714583518ce5febadc29aa4e3a41b7156378
Author: Viraj Jasani <vj...@apache.org>
AuthorDate: Fri Mar 6 13:01:17 2020 +0530

    HBASE-23909 : list_regions raise correct error if it is being splitted/merged/transitioning (#1238)
    
    Signed-off-by: Wellington Ramos Chevreuil <wc...@apache.org>
    Signed-off-by: Peter Somogyi <ps...@apache.org>
    Signed-off-by: Jan Hentschel <ja...@ultratendency.com>
---
 hbase-shell/src/main/ruby/shell/commands/list_regions.rb | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/hbase-shell/src/main/ruby/shell/commands/list_regions.rb b/hbase-shell/src/main/ruby/shell/commands/list_regions.rb
index 5ad6f1a..96581f1 100644
--- a/hbase-shell/src/main/ruby/shell/commands/list_regions.rb
+++ b/hbase-shell/src/main/ruby/shell/commands/list_regions.rb
@@ -208,7 +208,12 @@ EOF
 
       def get_regions_for_server(regions_for_table, server_name)
         regions_for_table.select do |hregion|
-          accept_server_name? server_name, hregion.getServerName.toString
+          actual_server_name = hregion.getServerName
+          if actual_server_name == nil
+            raise "Some regions might be splitting or merging or transitioning due to other" \
+            " reasons"
+          end
+          accept_server_name? server_name, actual_server_name.toString
         end
       end