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

Ted Yu commented on HBASE-6896:
-------------------------------

Patch looks good.
Should subject of this JIRA be changed to reflect the fact that socket timeout 
is handled ?
{code}
+              // just the response isn't gone through.  So we will retry to
{code}
"isn't gone" -> "didn't go"
{code}
+                  + " has timed out, but the region might"
{code}
Can you include region name in the above log ? In that case, loop index k 
should be declared outside the loop.

                
> sync bulk and regular assigment handling if possible
> ----------------------------------------------------
>
>                 Key: HBASE-6896
>                 URL: https://issues.apache.org/jira/browse/HBASE-6896
>             Project: HBase
>          Issue Type: Bug
>          Components: Region Assignment
>            Reporter: Jimmy Xiang
>            Assignee: Jimmy Xiang
>            Priority: Minor
>         Attachments: trunk-6896.patch
>
>
> Currently, for bulk assignment, when a region is already in transition, it 
> thinks the region is opened. However, in regular assignment, it throws 
> RegionAlreadyInTransitionException.
> In regular assignment, in case of RegionAlreadyInTransitionException, it 
> tries to call openRegion again and again without change the region plan, ZK 
> offline node,
> till the region is out of transition.
> We may need to sync them up and make sure bulk assignment does the same in 
> this case.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to