Return-Path: X-Original-To: apmail-hbase-commits-archive@www.apache.org Delivered-To: apmail-hbase-commits-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id B85E1184F2 for ; Tue, 10 Nov 2015 23:15:10 +0000 (UTC) Received: (qmail 68429 invoked by uid 500); 10 Nov 2015 23:15:10 -0000 Delivered-To: apmail-hbase-commits-archive@hbase.apache.org Received: (qmail 68387 invoked by uid 500); 10 Nov 2015 23:15:10 -0000 Mailing-List: contact commits-help@hbase.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@hbase.apache.org Delivered-To: mailing list commits@hbase.apache.org Received: (qmail 68378 invoked by uid 99); 10 Nov 2015 23:15:10 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 10 Nov 2015 23:15:10 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 74FF2E0514; Tue, 10 Nov 2015 23:15:10 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: jmhsieh@apache.org To: commits@hbase.apache.org Message-Id: <8563b40520634d1fb6e707df3e8f3205@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: hbase git commit: HBASE-14780 Integration Test that run with chaos monkey must specify cfs Date: Tue, 10 Nov 2015 23:15:10 +0000 (UTC) Repository: hbase Updated Branches: refs/heads/branch-1 1d29d8c00 -> f782cab42 HBASE-14780 Integration Test that run with chaos monkey must specify cfs Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/f782cab4 Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/f782cab4 Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/f782cab4 Branch: refs/heads/branch-1 Commit: f782cab42156291bfe6596666e5922ff53d80c5a Parents: 1d29d8c Author: Jonathan M Hsieh Authored: Fri Nov 6 11:39:51 2015 -0800 Committer: Jonathan M Hsieh Committed: Tue Nov 10 15:09:35 2015 -0800 ---------------------------------------------------------------------- .../java/org/apache/hadoop/hbase/IntegrationTestBase.java | 8 ++++++++ .../hadoop/hbase/IntegrationTestDDLMasterFailover.java | 4 ++-- .../hadoop/hbase/IntegrationTestRegionReplicaPerf.java | 4 +++- .../hadoop/hbase/mapreduce/IntegrationTestBulkLoad.java | 5 ++++- .../mapreduce/IntegrationTestTableSnapshotInputFormat.java | 8 +++----- 5 files changed, 20 insertions(+), 9 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/f782cab4/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestBase.java ---------------------------------------------------------------------- diff --git a/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestBase.java b/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestBase.java index 097673a..3929524 100644 --- a/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestBase.java +++ b/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestBase.java @@ -203,7 +203,15 @@ public abstract class IntegrationTestBase extends AbstractHBaseTool { public abstract int runTestFromCommandLine() throws Exception; + /** + * Provides the name of the table that is protected from random Chaos monkey activity + * @return table to not delete. + */ public abstract TableName getTablename(); + /** + * Provides the name of the CFs that are protected from random Chaos monkey activity (alter) + * @return set of cf names to protect. + */ protected abstract Set getColumnFamilies(); } http://git-wip-us.apache.org/repos/asf/hbase/blob/f782cab4/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestDDLMasterFailover.java ---------------------------------------------------------------------- diff --git a/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestDDLMasterFailover.java b/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestDDLMasterFailover.java index 3ba7818..dffde55 100644 --- a/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestDDLMasterFailover.java +++ b/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestDDLMasterFailover.java @@ -954,12 +954,12 @@ public class IntegrationTestDDLMasterFailover extends IntegrationTestBase { @Override public TableName getTablename() { - return null; + return null; // This test is not inteded to run with stock Chaos Monkey } @Override protected Set getColumnFamilies() { - return null; + return null; // This test is not inteded to run with stock Chaos Monkey } public static void main(String[] args) throws Exception { http://git-wip-us.apache.org/repos/asf/hbase/blob/f782cab4/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestRegionReplicaPerf.java ---------------------------------------------------------------------- diff --git a/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestRegionReplicaPerf.java b/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestRegionReplicaPerf.java index 3c29f4c..d05e039 100644 --- a/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestRegionReplicaPerf.java +++ b/hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestRegionReplicaPerf.java @@ -19,6 +19,7 @@ package org.apache.hadoop.hbase; import com.google.common.base.Objects; +import com.google.common.collect.Sets; import com.yammer.metrics.core.Histogram; import org.apache.commons.cli.CommandLine; import org.apache.commons.logging.Log; @@ -33,6 +34,7 @@ import org.apache.hadoop.hbase.client.Admin; import org.apache.hadoop.hbase.ipc.RpcClient; import org.apache.hadoop.hbase.regionserver.DisabledRegionSplitPolicy; import org.apache.hadoop.hbase.testclassification.IntegrationTests; +import org.apache.hadoop.hbase.util.Bytes; import org.apache.hadoop.hbase.util.YammerHistogramUtils; import org.apache.hadoop.mapreduce.Counters; import org.apache.hadoop.mapreduce.Job; @@ -234,7 +236,7 @@ public class IntegrationTestRegionReplicaPerf extends IntegrationTestBase { @Override protected Set getColumnFamilies() { - return null; + return Sets.newHashSet(Bytes.toString(PerformanceEvaluation.FAMILY_NAME)); } /** Compute the mean of the given {@code stat} from a timing results. */ http://git-wip-us.apache.org/repos/asf/hbase/blob/f782cab4/hbase-it/src/test/java/org/apache/hadoop/hbase/mapreduce/IntegrationTestBulkLoad.java ---------------------------------------------------------------------- diff --git a/hbase-it/src/test/java/org/apache/hadoop/hbase/mapreduce/IntegrationTestBulkLoad.java b/hbase-it/src/test/java/org/apache/hadoop/hbase/mapreduce/IntegrationTestBulkLoad.java index b5f0360..b6a6bfe 100644 --- a/hbase-it/src/test/java/org/apache/hadoop/hbase/mapreduce/IntegrationTestBulkLoad.java +++ b/hbase-it/src/test/java/org/apache/hadoop/hbase/mapreduce/IntegrationTestBulkLoad.java @@ -22,6 +22,7 @@ import static org.junit.Assert.assertEquals; import com.google.common.base.Joiner; +import com.google.common.collect.Sets; import org.apache.commons.cli.CommandLine; import org.apache.commons.lang.RandomStringUtils; import org.apache.commons.logging.Log; @@ -79,6 +80,7 @@ import java.io.DataInput; import java.io.DataOutput; import java.io.IOException; import java.util.ArrayList; +import java.util.HashSet; import java.util.List; import java.util.Map; import java.util.Random; @@ -778,7 +780,8 @@ public class IntegrationTestBulkLoad extends IntegrationTestBase { @Override protected Set getColumnFamilies() { - return null; + return Sets.newHashSet(Bytes.toString(CHAIN_FAM) , Bytes.toString(DATA_FAM), + Bytes.toString(SORT_FAM)); } public static void main(String[] args) throws Exception { http://git-wip-us.apache.org/repos/asf/hbase/blob/f782cab4/hbase-it/src/test/java/org/apache/hadoop/hbase/mapreduce/IntegrationTestTableSnapshotInputFormat.java ---------------------------------------------------------------------- diff --git a/hbase-it/src/test/java/org/apache/hadoop/hbase/mapreduce/IntegrationTestTableSnapshotInputFormat.java b/hbase-it/src/test/java/org/apache/hadoop/hbase/mapreduce/IntegrationTestTableSnapshotInputFormat.java index 55da93e..1a152e8 100644 --- a/hbase-it/src/test/java/org/apache/hadoop/hbase/mapreduce/IntegrationTestTableSnapshotInputFormat.java +++ b/hbase-it/src/test/java/org/apache/hadoop/hbase/mapreduce/IntegrationTestTableSnapshotInputFormat.java @@ -174,15 +174,13 @@ public class IntegrationTestTableSnapshotInputFormat extends IntegrationTestBase return 0; } - @Override // CM is not intended to be run with this test + @Override // Chaos Monkey is not intended to be run with this test public TableName getTablename() { return null; } - @Override - protected Set getColumnFamilies() { - return null; - } + @Override // Chaos Monkey is not inteded to be run with this test + protected Set getColumnFamilies() { return null; } public static void main(String[] args) throws Exception { Configuration conf = HBaseConfiguration.create();