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

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


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

commit 5e67e6badcc5a00922bca2ff471f1b4b221734d1
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 dc77ae7..f63ff78 100644
--- a/hbase-shell/src/main/ruby/shell/commands/list_regions.rb
+++ b/hbase-shell/src/main/ruby/shell/commands/list_regions.rb
@@ -215,7 +215,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