diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/SnapshotDescription.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/SnapshotDescription.java index 6e31979d181..bbc6a503a3e 100644 --- a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/SnapshotDescription.java +++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/SnapshotDescription.java @@ -38,6 +38,8 @@ public class SnapshotDescription { private final long ttl; private final int version; + private final long maxFileSize; + public SnapshotDescription(String name) { this(name, null); } @@ -90,14 +92,17 @@ public class SnapshotDescription { this.snapShotType = type; this.owner = owner; this.creationTime = creationTime; - this.ttl = getTtlFromSnapshotProps(snapshotProps); + this.ttl = getLongFromSnapshotProps(snapshotProps, "TTL"); this.version = version; + this.maxFileSize = getLongFromSnapshotProps(snapshotProps, TableDescriptorBuilder.MAX_FILESIZE); } - private long getTtlFromSnapshotProps(Map snapshotProps) { - return MapUtils.getLongValue(snapshotProps, "TTL", -1); + private long getLongFromSnapshotProps(Map snapshotProps, String property) { + return MapUtils.getLongValue(snapshotProps, property, -1); } + + /** * SnapshotDescription Parameterized Constructor * @@ -144,6 +149,8 @@ public class SnapshotDescription { return this.version; } + public long getMaxFileSize() { return maxFileSize; } + @Override public String toString() { return new ToStringBuilder(this) @@ -154,6 +161,7 @@ public class SnapshotDescription { .append("creationTime", creationTime) .append("ttl", ttl) .append("version", version) + .append("maxFileSize", maxFileSize) .toString(); } } diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/shaded/protobuf/ProtobufUtil.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/shaded/protobuf/ProtobufUtil.java index 462ffb012d9..ed001c3093b 100644 --- a/hbase-client/src/main/java/org/apache/hadoop/hbase/shaded/protobuf/ProtobufUtil.java +++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/shaded/protobuf/ProtobufUtil.java @@ -3092,6 +3092,9 @@ public final class ProtobufUtil { if (snapshotDesc.getVersion() != -1) { builder.setVersion(snapshotDesc.getVersion()); } + if (snapshotDesc.getMaxFileSize() != -1) { + builder.setMaxFileSize(snapshotDesc.getMaxFileSize()); + } builder.setType(ProtobufUtil.createProtosSnapShotDescType(snapshotDesc.getType())); return builder.build(); } @@ -3107,6 +3110,7 @@ public final class ProtobufUtil { createSnapshotDesc(SnapshotProtos.SnapshotDescription snapshotDesc) { final Map snapshotProps = new HashMap<>(); snapshotProps.put("TTL", snapshotDesc.getTtl()); + snapshotProps.put(TableDescriptorBuilder.MAX_FILESIZE, snapshotDesc.getMaxFileSize()); return new SnapshotDescription(snapshotDesc.getName(), snapshotDesc.hasTable() ? TableName.valueOf(snapshotDesc.getTable()) : null, createSnapshotType(snapshotDesc.getType()), snapshotDesc.getOwner(), diff --git a/hbase-protocol-shaded/src/main/protobuf/server/Snapshot.proto b/hbase-protocol-shaded/src/main/protobuf/server/Snapshot.proto index 0239d0c84a9..9d5a0afd029 100644 --- a/hbase-protocol-shaded/src/main/protobuf/server/Snapshot.proto +++ b/hbase-protocol-shaded/src/main/protobuf/server/Snapshot.proto @@ -45,6 +45,7 @@ message SnapshotDescription { optional string owner = 6; optional UsersAndPermissions users_and_permissions = 7; optional int64 ttl = 8 [default = 0]; + optional int64 max_file_size = 9 [default = 0]; } message SnapshotFileInfo { diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/SnapshotManager.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/SnapshotManager.java index 30107790677..cdacff7607a 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/SnapshotManager.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/SnapshotManager.java @@ -150,6 +150,10 @@ public class SnapshotManager extends MasterProcedureManager implements Stoppable /** number of current operations running on the master */ public static final int SNAPSHOT_POOL_THREADS_DEFAULT = 1; + /** Conf key for preserving original max file size configs */ + public static final String SNAPSHOT_MAX_FILE_SIZE_PRESERVE = + "hbase.snapshot.max.filesize.preserve"; + private boolean stopped; private MasterServices master; // Needed by TableEventHandlers private ProcedureCoordinator coordinator; diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/TakeSnapshotHandler.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/TakeSnapshotHandler.java index 7c9496b6681..5ff8a499ce5 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/TakeSnapshotHandler.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/TakeSnapshotHandler.java @@ -17,7 +17,6 @@ */ package org.apache.hadoop.hbase.master.snapshot; -import java.io.FileNotFoundException; import java.io.IOException; import java.util.HashSet; import java.util.List; @@ -31,6 +30,7 @@ import org.apache.hadoop.hbase.ServerName; import org.apache.hadoop.hbase.TableName; import org.apache.hadoop.hbase.client.RegionInfo; import org.apache.hadoop.hbase.client.TableDescriptor; +import org.apache.hadoop.hbase.client.TableDescriptorBuilder; import org.apache.hadoop.hbase.errorhandling.ForeignException; import org.apache.hadoop.hbase.errorhandling.ForeignExceptionDispatcher; import org.apache.hadoop.hbase.errorhandling.ForeignExceptionSnare; @@ -139,12 +139,17 @@ public abstract class TakeSnapshotHandler extends EventHandler implements Snapsh } private TableDescriptor loadTableDescriptor() - throws FileNotFoundException, IOException { + throws IOException { TableDescriptor htd = this.master.getTableDescriptors().get(snapshotTable); if (htd == null) { throw new IOException("TableDescriptor missing for " + snapshotTable); } + if (htd.getMaxFileSize()==-1 && + this.snapshot.getMaxFileSize()>0) { + htd = TableDescriptorBuilder.newBuilder(htd).setValue(TableDescriptorBuilder.MAX_FILESIZE, + Long.toString(this.snapshot.getMaxFileSize())).build(); + } return htd; } diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/snapshot/TestTakeSnapshotHandler.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/snapshot/TestTakeSnapshotHandler.java new file mode 100644 index 00000000000..9000254db31 --- /dev/null +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/snapshot/TestTakeSnapshotHandler.java @@ -0,0 +1,111 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * with the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package org.apache.hadoop.hbase.master.snapshot; + +import static org.junit.Assert.assertEquals; + +import java.util.HashMap; +import java.util.Map; +import org.apache.hadoop.hbase.HBaseClassTestRule; +import org.apache.hadoop.hbase.HBaseTestingUtility; +import org.apache.hadoop.hbase.TableName; +import org.apache.hadoop.hbase.client.ColumnFamilyDescriptorBuilder; +import org.apache.hadoop.hbase.client.Put; +import org.apache.hadoop.hbase.client.Table; +import org.apache.hadoop.hbase.client.TableDescriptor; +import org.apache.hadoop.hbase.client.TableDescriptorBuilder; +import org.apache.hadoop.hbase.testclassification.MediumTests; +import org.apache.hadoop.hbase.util.Bytes; +import org.junit.After; +import org.junit.Before; +import org.junit.ClassRule; +import org.junit.Rule; +import org.junit.Test; +import org.junit.experimental.categories.Category; +import org.junit.rules.TestName; + + +/** + * Unfortunately, couldn't test TakeSnapshotHandler using mocks, because it relies on TableLock, + * which is tightly coupled to LockManager and LockProcedure classes, which are both final and + * prevents us from mocking its behaviour. Looks like an overkill having to emulate a + * whole cluster run for such a small optional property behaviour. + */ +@Category({ MediumTests.class}) +public class TestTakeSnapshotHandler { + + private static HBaseTestingUtility UTIL; + + @ClassRule + public static final HBaseClassTestRule CLASS_RULE = + HBaseClassTestRule.forClass(TestTakeSnapshotHandler.class); + + @Rule + public TestName name = new TestName(); + + + @Before + public void setup() { + UTIL = new HBaseTestingUtility(); + } + + public TableDescriptor createTableInsertDataAndTakeSnapshot(Map snapshotProps) + throws Exception { + TableDescriptor descriptor = + TableDescriptorBuilder.newBuilder(TableName.valueOf(name.getMethodName())) + .setColumnFamily( + ColumnFamilyDescriptorBuilder.newBuilder(Bytes.toBytes("f")).build()).build(); + UTIL.getConnection().getAdmin().createTable(descriptor); + Table table = UTIL.getConnection().getTable(descriptor.getTableName()); + Put put = new Put(Bytes.toBytes("1")); + put.addColumn(Bytes.toBytes("f"), Bytes.toBytes("1"), Bytes.toBytes("v1")); + table.put(put); + String snapName = "snap"+name.getMethodName(); + UTIL.getAdmin().snapshot(snapName, descriptor.getTableName(), snapshotProps); + TableName cloned = TableName.valueOf(name.getMethodName() + "clone"); + UTIL.getAdmin().cloneSnapshot(snapName, cloned); + return descriptor; + } + + @Test + public void testPreparePreserveMaxFileSizeEnabled() throws Exception { + UTIL.startMiniCluster(); + Map snapshotProps = new HashMap<>(); + snapshotProps.put(TableDescriptorBuilder.MAX_FILESIZE, Long.parseLong("21474836480")); + TableDescriptor descriptor = createTableInsertDataAndTakeSnapshot(snapshotProps); + TableName cloned = TableName.valueOf(name.getMethodName() + "clone"); + assertEquals(-1, + UTIL.getAdmin().getDescriptor(descriptor.getTableName()).getMaxFileSize()); + assertEquals(21474836480L, UTIL.getAdmin().getDescriptor(cloned).getMaxFileSize()); + } + + @Test + public void testPreparePreserveMaxFileSizeDisabled() throws Exception { + UTIL.startMiniCluster(); + TableDescriptor descriptor = createTableInsertDataAndTakeSnapshot(null); + TableName cloned = TableName.valueOf(name.getMethodName() + "clone"); + assertEquals(-1, + UTIL.getAdmin().getDescriptor(descriptor.getTableName()).getMaxFileSize()); + assertEquals(-1, UTIL.getAdmin().getDescriptor(cloned).getMaxFileSize()); + } + + @After + public void shutdown() throws Exception { + UTIL.shutdownMiniCluster(); + } +} diff --git a/hbase-shell/src/main/ruby/hbase/admin.rb b/hbase-shell/src/main/ruby/hbase/admin.rb index 33007ee9cd6..a546605c621 100644 --- a/hbase-shell/src/main/ruby/hbase/admin.rb +++ b/hbase-shell/src/main/ruby/hbase/admin.rb @@ -1206,6 +1206,9 @@ module Hbase ttl = ttl ? ttl.to_java(:long) : -1 snapshot_props = java.util.HashMap.new snapshot_props.put("TTL", ttl) + max_filesize = arg[MAX_FILESIZE] + max_filesize = max_filesize ? max_filesize.to_java(:long) : -1 + snapshot_props.put("MAX_FILESIZE", max_filesize) if arg[SKIP_FLUSH] == true @admin.snapshot(snapshot_name, table_name, org.apache.hadoop.hbase.client.SnapshotType::SKIPFLUSH, snapshot_props) diff --git a/hbase-shell/src/main/ruby/shell/commands/snapshot.rb b/hbase-shell/src/main/ruby/shell/commands/snapshot.rb index c591e12b5df..99844947081 100644 --- a/hbase-shell/src/main/ruby/shell/commands/snapshot.rb +++ b/hbase-shell/src/main/ruby/shell/commands/snapshot.rb @@ -24,7 +24,7 @@ module Shell Take a snapshot of specified table. Examples: hbase> snapshot 'sourceTable', 'snapshotName' - hbase> snapshot 'namespace:sourceTable', 'snapshotName', {SKIP_FLUSH => true} + hbase> snapshot 'namespace:sourceTable', 'snapshotName', {SKIP_FLUSH => true, MAX_FILESIZE => 21474836480} EOF end diff --git a/src/main/asciidoc/_chapters/ops_mgt.adoc b/src/main/asciidoc/_chapters/ops_mgt.adoc index a5bdeb708a5..c383c44a81c 100644 --- a/src/main/asciidoc/_chapters/ops_mgt.adoc +++ b/src/main/asciidoc/_chapters/ops_mgt.adoc @@ -3167,6 +3167,21 @@ providing default TTL in sec for key: `hbase.master.snapshot.ttl`. Value 0 for this config indicates TTL: FOREVER +.Take a snapshot with custom MAX_FILESIZE + +Optionally, snapshots can be created with a custom max file size configuration that will be +used by cloned tables, instead of the global `hbase.hregion.max.filesize` configuration property. +This is mostly useful when exporting snapshots between different clusters. If the HBase cluster where +the snapshot is originally taken has a much larger value set for `hbase.hregion.max.filesize` than +one or more clusters where the snapshot is being exported to, a storm of region splits may occur when +restoring the snapshot on destination clusters. Specifying `MAX_FILESIZE` on properties passed to +`snapshot` command will save informed value into the table's `MAX_FILESIZE` +decriptor at snapshot creation time. If the table already defines `MAX_FILESIZE` descriptor, +this property would be ignored and have no effect. + +---- +snapshot 'table01', 'snap01', {MAX_FILESIZE => 21474836480} +---- .Enable/Disable Snapshot Auto Cleanup on running cluster: