[
https://issues.apache.org/jira/browse/HBASE-8160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13608960#comment-13608960
]
Jonathan Hsieh commented on HBASE-8160:
---------------------------------------
The other uses of checkInitialized just wrap with ServiceException (no extra
UnknownRegionException wrapping on init check fails). Stick to the pattern
(and remove comment)?
Other methods have a this structural pattern, consider using it in moveRegion
as well?
{code}
ProtoResponse method(ProtoRequest) {
...
actualMethod()
...
}
retType actualMethod(arg, arg, arg) {
..
checkInitialized()
.. do stuff
}
{code}
> HMaster#move doesn't check if master initialized
> ------------------------------------------------
>
> Key: HBASE-8160
> URL: https://issues.apache.org/jira/browse/HBASE-8160
> Project: HBase
> Issue Type: Bug
> Components: master
> Reporter: Jimmy Xiang
> Assignee: Jimmy Xiang
> Priority: Trivial
> Fix For: 0.95.0, 0.98.0, 0.94.7
>
> Attachments: 0.94-8160.patch, trunk-8160.patch
>
>
> Before the master is initialized, if a move request comes in, it could cause
> the master to abort, if the region was already in transition before the
> master started up.
--
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
