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 <[email protected]>
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 <[email protected]>
    Signed-off-by: Peter Somogyi <[email protected]>
    Signed-off-by: Jan Hentschel <[email protected]>
---
 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
 

Reply via email to