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

Ted Yu commented on HBASE-7858:
-------------------------------

Looking at other methods which access this.handler, such as 
snapshotEnabledTable(), they're declared synchronized.

getTakeSnapshotHandler() is called by isSnapshotDone() which is not 
synchronized, hence the change.
                
> cleanup before merging snapshots branch to trunk
> ------------------------------------------------
>
>                 Key: HBASE-7858
>                 URL: https://issues.apache.org/jira/browse/HBASE-7858
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Ted Yu
>            Assignee: Ted Yu
>         Attachments: 7858-v1.txt, 7858-v2.txt, 7858-v3.txt, 7858-v4.txt, 
> 7858-v5.txt, 7858-v6.txt, 7858-v7.txt, 7858-v8.txt
>
>
> There have been a lot of review comments from 
> https://reviews.apache.org/r/9416
> Since our goal of merging snapshot feature to trunk would preserve revision 
> history, a separate JIRA is needed.

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