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

Jonathan Hsieh commented on HBASE-8341:
---------------------------------------

+1. 

[~enis] Thanks for the explanation.  I'd prefer it if the TableDescriptor fix 
was a separate patch as it is a different and not directly related problem than 
the title.  When committing you may want to have two reasons in the fix, or 
commit as two separate patches?
                
> RestoreSnapshotHandler.prepare() is not called by SnapshotManager  
> -------------------------------------------------------------------
>
>                 Key: HBASE-8341
>                 URL: https://issues.apache.org/jira/browse/HBASE-8341
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Enis Soztutar
>            Assignee: Enis Soztutar
>             Fix For: 0.98.0, 0.95.1
>
>         Attachments: hbase-8341_v1.patch
>
>
> In HBASE-7848, we added table lock to enabled/disabled snapshot handlers, and 
> fixed SnapshotManager to call CloneSnapshotHandler.prepare() in HBASE-7957. 
> It seems that we overlooked the RestoreSnapshotHandler.prepare(). In this 
> issue we should fix that so that we acquire the table lock in restore 
> snapshot. 

--
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