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

Ted Yu commented on HBASE-17938:
--------------------------------

Somehow review board doesn't accept my review comments.
{code}
      conn = ConnectionFactory.createConnection(getConf());
{code}
Where is conn released ?
{code}
      firstBackup = savedStartCode == null || Long.parseLong(savedStartCode) == 
0L;
{code}
What if a second client comes and sees the savedStartCode as zero (written by 
line 130) ?

There is duplicate code between executeForTesting() and execute(). Extract 
common code.


> General fault - tolerance framework for backup/restore operations
> -----------------------------------------------------------------
>
>                 Key: HBASE-17938
>                 URL: https://issues.apache.org/jira/browse/HBASE-17938
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Vladimir Rodionov
>            Assignee: Vladimir Rodionov
>             Fix For: 2.0.0
>
>         Attachments: HBASE-17938-v1.patch, HBASE-17938-v2.patch, 
> HBASE-17938-v3.patch, HBASE-17938-v4.patch
>
>
> The framework must take care of all general types of failures during backup/ 
> restore and restore system to the original state in case of a failure.
> That won't solve all the possible issues  but we have a separate JIRAs for 
> them as a sub-tasks of HBASE-15277



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to