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

Ted Yu commented on HBASE-15411:
--------------------------------

Here are the existing Util classes in hbase-client:
{code}
hbase-client//src/main/java/org/apache/hadoop/hbase/client/RegionReplicaUtil.java
hbase-client//src/main/java/org/apache/hadoop/hbase/client/ResultStatsUtil.java
hbase-client//src/main/java/org/apache/hadoop/hbase/exceptions/ClientExceptionsUtil.java
hbase-client//src/main/java/org/apache/hadoop/hbase/ipc/IPCUtil.java
hbase-client//src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufUtil.java
hbase-client//src/main/java/org/apache/hadoop/hbase/quotas/QuotaTableUtil.java
hbase-client//src/main/java/org/apache/hadoop/hbase/security/EncryptionUtil.java
hbase-client//src/main/java/org/apache/hadoop/hbase/security/SaslUtil.java
hbase-client//src/main/java/org/apache/hadoop/hbase/security/SecureBulkLoadUtil.java
hbase-client//src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
hbase-client//src/test/java/org/apache/hadoop/hbase/exceptions/TestClientExceptionsUtil.java
hbase-client//src/test/java/org/apache/hadoop/hbase/ipc/TestIPCUtil.java
hbase-client//src/test/java/org/apache/hadoop/hbase/security/TestEncryptionUtil.java
hbase-client//src/test/java/org/apache/hadoop/hbase/security/TestSaslUtil.java
hbase-client//src/test/java/org/apache/hadoop/hbase/zookeeper/TestZKUtil.java
{code}
I don't think the waiting logic should go to any of the above class.

Maybe create 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/AdminUtil.java ?

> Rewrite backup with Procedure V2
> --------------------------------
>
>                 Key: HBASE-15411
>                 URL: https://issues.apache.org/jira/browse/HBASE-15411
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Ted Yu
>            Assignee: Ted Yu
>         Attachments: 15411-v1.txt, 15411-v11.txt, 15411-v12.txt, 
> 15411-v13.txt, 15411-v14.txt, 15411-v15.txt, 15411-v16.txt, 15411-v18.txt, 
> 15411-v22.txt, 15411-v3.txt, 15411-v5.txt, 15411-v6.txt, 15411-v7.txt, 
> 15411-v9.txt, FullTableBackupProcedure.java
>
>
> Currently full / incremental backup is driven by BackupHandler (see call() 
> method for flow).
> This issue is to rewrite the flow using Procedure V2.
> States (enum) for full / incremental backup would be introduced in 
> Backup.proto which correspond to the steps performed in BackupHandler#call().
> executeFromState() would pace the backup based on the current state.
> serializeStateData() / deserializeStateData() would be used to persist state 
> into procedure WAL.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to