[ 
https://issues.apache.org/jira/browse/HBASE-6012?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13290254#comment-13290254
 ] 

Zhihong Ted Yu commented on HBASE-6012:
---------------------------------------

{code}
+          List<RegionOpeningState> regionOpeningStateList = this.serverManager
+              .sendRegionOpen(destination, regions);
+          for (int i = 0; i < regionOpeningStateList.size(); i++) {
{code}
Should we check whether the return from sendRegionOpen() is null ?

In ServerShutdownHandler.java:
{code}
+                if(rit != null){
+                  //clean zk node
+                  try{
+                    ZKAssign.deleteNodeFailSilent(services.getZooKeeper(), 
e.getKey());
{code}
Log statement should be added that reveals the value of rit.

                
> Handling RegionOpeningState for bulk assign since SSH using
> -----------------------------------------------------------
>
>                 Key: HBASE-6012
>                 URL: https://issues.apache.org/jira/browse/HBASE-6012
>             Project: HBase
>          Issue Type: Bug
>    Affects Versions: 0.96.0
>            Reporter: chunhui shen
>            Assignee: chunhui shen
>             Fix For: 0.96.0
>
>         Attachments: HBASE-6012.patch, HBASE-6012v2.patch, HBASE-6012v3.patch
>
>
> Since HBASE-5914, we using bulk assign for SSH
> But in the bulk assign case if we get an ALREADY_OPENED case there is no one 
> to clear the znode created by bulk assign. 
> Another thing, when RS opening a list of regions, if one region is already in 
> transition, it will throw RegionAlreadyInTransitionException and stop opening 
> other regions.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to