Return-Path: X-Original-To: apmail-accumulo-commits-archive@www.apache.org Delivered-To: apmail-accumulo-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 9FF221771E for ; Tue, 4 Nov 2014 19:17:08 +0000 (UTC) Received: (qmail 12028 invoked by uid 500); 4 Nov 2014 19:17:08 -0000 Delivered-To: apmail-accumulo-commits-archive@accumulo.apache.org Received: (qmail 11996 invoked by uid 500); 4 Nov 2014 19:17:08 -0000 Mailing-List: contact commits-help@accumulo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@accumulo.apache.org Delivered-To: mailing list commits@accumulo.apache.org Received: (qmail 11987 invoked by uid 99); 4 Nov 2014 19:17:08 -0000 Received: from tyr.zones.apache.org (HELO tyr.zones.apache.org) (140.211.11.114) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 04 Nov 2014 19:17:08 +0000 Received: by tyr.zones.apache.org (Postfix, from userid 65534) id 353DD8AE9EA; Tue, 4 Nov 2014 19:17:08 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: elserj@apache.org To: commits@accumulo.apache.org Date: Tue, 04 Nov 2014 19:17:11 -0000 Message-Id: <8239872aaa534514b1c6dd8c48480313@git.apache.org> In-Reply-To: <192ea1824b3e4ba9aba653749ae92441@git.apache.org> References: <192ea1824b3e4ba9aba653749ae92441@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: [5/7] git commit: ACCUMULO-3243 Add missing defaultTimeoutSeconds and remove method-level timeout ACCUMULO-3243 Add missing defaultTimeoutSeconds and remove method-level timeout Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/230a938a Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/230a938a Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/230a938a Branch: refs/heads/master Commit: 230a938a0a91ae640e185406de69b51c92ccb6c1 Parents: 73f3759 Author: Josh Elser Authored: Tue Nov 4 13:56:34 2014 -0500 Committer: Josh Elser Committed: Tue Nov 4 14:16:09 2014 -0500 ---------------------------------------------------------------------- .../test/RewriteTabletDirectoriesIT.java | 23 ++++++++++++-------- .../org/apache/accumulo/test/ShellServerIT.java | 2 +- 2 files changed, 15 insertions(+), 10 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/accumulo/blob/230a938a/test/src/test/java/org/apache/accumulo/test/RewriteTabletDirectoriesIT.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/test/RewriteTabletDirectoriesIT.java b/test/src/test/java/org/apache/accumulo/test/RewriteTabletDirectoriesIT.java index f2efa11..fe233ee 100644 --- a/test/src/test/java/org/apache/accumulo/test/RewriteTabletDirectoriesIT.java +++ b/test/src/test/java/org/apache/accumulo/test/RewriteTabletDirectoriesIT.java @@ -16,6 +16,10 @@ */ package org.apache.accumulo.test; +import static org.apache.accumulo.core.metadata.schema.MetadataSchema.TabletsSection.ServerColumnFamily.DIRECTORY_COLUMN; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertTrue; + import java.io.BufferedOutputStream; import java.io.File; import java.io.FileOutputStream; @@ -46,13 +50,14 @@ import org.apache.hadoop.fs.RawLocalFileSystem; import org.apache.hadoop.io.Text; import org.junit.Test; -import static org.apache.accumulo.core.metadata.schema.MetadataSchema.TabletsSection.ServerColumnFamily.DIRECTORY_COLUMN; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertTrue; - // ACCUMULO-3263 public class RewriteTabletDirectoriesIT extends ConfigurableMacIT { + @Override + public int defaultTimeoutSeconds() { + return 4 * 60; + } + private Path v1, v2; @Override @@ -72,7 +77,7 @@ public class RewriteTabletDirectoriesIT extends ConfigurableMacIT { super.configure(cfg, hadoopCoreSite); } - @Test(timeout = 4 * 60 * 1000) + @Test public void test() throws Exception { Connector c = getConnector(); c.securityOperations().grantTablePermission(c.whoami(), MetadataTable.NAME, TablePermission.WRITE); @@ -88,7 +93,7 @@ public class RewriteTabletDirectoriesIT extends ConfigurableMacIT { } bw.close(); c.tableOperations().addSplits(tableName, splits); - + BatchScanner scanner = c.createBatchScanner(MetadataTable.NAME, Authorizations.EMPTY, 1); DIRECTORY_COLUMN.fetch(scanner); String tableId = c.tableOperations().tableIdMap().get(tableName); @@ -109,10 +114,10 @@ public class RewriteTabletDirectoriesIT extends ConfigurableMacIT { } bw.close(); assertEquals(splits.size() + 1, count); - + // This should fail: only one volume assertEquals(1, cluster.exec(RandomizeVolumes.class, "-z", cluster.getZooKeepers(), "-i", c.getInstance().getInstanceName(), "-t", tableName).waitFor()); - + cluster.stop(); // add the 2nd volume @@ -122,7 +127,7 @@ public class RewriteTabletDirectoriesIT extends ConfigurableMacIT { BufferedOutputStream fos = new BufferedOutputStream(new FileOutputStream(new File(cluster.getConfig().getConfDir(), "accumulo-site.xml"))); conf.writeXml(fos); fos.close(); - + // initialize volume assertEquals(0, cluster.exec(Initialize.class, "--add-volumes").waitFor()); cluster.start(); http://git-wip-us.apache.org/repos/asf/accumulo/blob/230a938a/test/src/test/java/org/apache/accumulo/test/ShellServerIT.java ---------------------------------------------------------------------- diff --git a/test/src/test/java/org/apache/accumulo/test/ShellServerIT.java b/test/src/test/java/org/apache/accumulo/test/ShellServerIT.java index 50e2d2b..4457e70 100644 --- a/test/src/test/java/org/apache/accumulo/test/ShellServerIT.java +++ b/test/src/test/java/org/apache/accumulo/test/ShellServerIT.java @@ -828,7 +828,7 @@ public class ShellServerIT extends SimpleMacIT { ts.exec("deletetable -f " + table, true); } - @Test(timeout = 45000) + @Test public void help() throws Exception { ts.exec("help -np", true, "Help Commands", true); ts.exec("?", true, "Help Commands", true);