From d22c6de6484607d0082a48b686c706d8e73650f1 Mon Sep 17 00:00:00 2001 From: Ankit Singhal Date: Fri, 29 Jun 2018 16:32:17 -0400 Subject: [PATCH] HBASE-20825 Fix pre and post hooks of CloneSnapshot and RestoreSnapshot for Access checks Signed-off-by: Josh Elser Signed-off-by: Ted Yu Signed-off-by: zhangduo --- .../apache/hadoop/hbase/master/snapshot/SnapshotManager.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) 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 1abd605f990..bc472d3d930 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 @@ -697,7 +697,7 @@ public class SnapshotManager extends MasterProcedureManager implements Stoppable TableDescriptor htd = TableDescriptorBuilder.copy(tableName, snapshotTableDesc); org.apache.hadoop.hbase.client.SnapshotDescription snapshotPOJO = null; if (cpHost != null) { - snapshotPOJO = ProtobufUtil.createSnapshotDesc(reqSnapshot); + snapshotPOJO = ProtobufUtil.createSnapshotDesc(snapshot); cpHost.preCloneSnapshot(snapshotPOJO, htd); } long procId; @@ -827,7 +827,7 @@ public class SnapshotManager extends MasterProcedureManager implements Stoppable // call Coprocessor pre hook org.apache.hadoop.hbase.client.SnapshotDescription snapshotPOJO = null; if (cpHost != null) { - snapshotPOJO = ProtobufUtil.createSnapshotDesc(reqSnapshot); + snapshotPOJO = ProtobufUtil.createSnapshotDesc(snapshot); cpHost.preRestoreSnapshot(snapshotPOJO, snapshotTableDesc); }