[
https://issues.apache.org/jira/browse/HBASE-17825?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16357357#comment-16357357
]
Josh Elser commented on HBASE-17825:
------------------------------------
{code}
+ if (fileName.indexOf(".tabledesc") > 0 ||
+ fileName.indexOf(".regioninfo") > 0 ) {
{code}
Do we have constants elsewhere we can use instead of the literal strings?
{code}
TableMapReduceUtil.addDependencyJarsForClasses(job.getConfiguration(),
-
org.apache.hbase.thirdparty.com.google.common.base.Preconditions.class);
+
org.apache.hbase.thirdparty.com.google.common.base.Preconditions.class);
{code}
{{TableMapReduceUtil.addHBaseDependencyJars}} should be doing this already for
you. I think you should be able to drop this right now, but maybe it's good to
preserve to prevent any breakage in the future? Not sure..
Also curious about the change in data directory that Ted pointed out.
> Backup: further optimizations
> -----------------------------
>
> Key: HBASE-17825
> URL: https://issues.apache.org/jira/browse/HBASE-17825
> Project: HBase
> Issue Type: Improvement
> Reporter: Vladimir Rodionov
> Assignee: Vladimir Rodionov
> Priority: Critical
> Labels: backup
> Fix For: 3.0.0
>
> Attachments: HBASE-17825-v1.patch
>
>
> Some phases of backup and restore can be optimized:
> # WALPlayer support for multiple tables
> # Run DistCp once per all tables during backup/restore
> The eventual goal:
> # 2 M/R jobs per backup/restore
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
