[
https://issues.apache.org/jira/browse/HBASE-15584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15321906#comment-15321906
]
Matteo Bertozzi commented on HBASE-15584:
-----------------------------------------
I don't see an answer to my question about why we need two system that do the
same thing and what is the advantage of the backupId over the procId alone.
if you just want to add an api that does abort(backupId) why the abort(procId)
is not enough? how do you find your backupId to pass to the abort()? is there
any information/helper that we can add to the listProcedures() that is
currently missing? how the user will find which is the backupId that he wants
to abort? I guess it will be similar to listProcedures().
can you provide an example with step by step operations that the user will do
to abort the backup? are they different from my comment above (20 may)? I still
don't have clear picture of what is the advantage/difference of the backupId,
so I can't help much at the moment.
> Revisit handling of BackupState#CANCELLED
> -----------------------------------------
>
> Key: HBASE-15584
> URL: https://issues.apache.org/jira/browse/HBASE-15584
> Project: HBase
> Issue Type: Sub-task
> Reporter: Ted Yu
> Priority: Minor
>
> During review of HBASE-15411, Enis made the following point:
> {code}
> nobody puts the backup in cancelled state. setCancelled() is not used. So if
> I abort a backup, who writes to the system table the new state?
> Not sure whether this is a phase 1 patch issue or due to this patch. We can
> open a new jira and address it there if you do not want to do it in this
> patch.
> Also maybe this should be named ABORTED rather than CANCELLED.
> {code}
> This issue is to decide whether this state should be kept (e.g. through
> notification from procedure V2 framework in response to abortion).
> If it is to be kept, the state should be renamed ABORTED.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
