You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by da...@apache.org on 2016/03/16 17:03:05 UTC

incubator-kudu git commit: [java tests] extend admin timeout and fix log4j.properties

Repository: incubator-kudu
Updated Branches:
  refs/heads/master 75c277b3b -> 830c3e4fe


[java tests] extend admin timeout and fix log4j.properties

Before this change I was consistently having test failures due to tablet
creation taking more than the default admin operation timeout.

Change-Id: Ie3b91abe65e07b4d7080bd4b9f9e2c904e505e9c
Reviewed-on: http://gerrit.cloudera.org:8080/2552
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <ad...@cloudera.com>


Project: http://git-wip-us.apache.org/repos/asf/incubator-kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-kudu/commit/830c3e4f
Tree: http://git-wip-us.apache.org/repos/asf/incubator-kudu/tree/830c3e4f
Diff: http://git-wip-us.apache.org/repos/asf/incubator-kudu/diff/830c3e4f

Branch: refs/heads/master
Commit: 830c3e4feb01528107a4bc4f67177959ac4dc33c
Parents: 75c277b
Author: Dan Burkert <da...@cloudera.com>
Authored: Tue Mar 15 13:36:24 2016 -0700
Committer: Dan Burkert <da...@cloudera.com>
Committed: Wed Mar 16 16:02:48 2016 +0000

----------------------------------------------------------------------
 .../src/test/java/org/kududb/client/BaseKuduTest.java            | 4 +++-
 java/kudu-client/src/test/resources/log4j.properties             | 4 ++--
 2 files changed, 5 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-kudu/blob/830c3e4f/java/kudu-client/src/test/java/org/kududb/client/BaseKuduTest.java
----------------------------------------------------------------------
diff --git a/java/kudu-client/src/test/java/org/kududb/client/BaseKuduTest.java b/java/kudu-client/src/test/java/org/kududb/client/BaseKuduTest.java
index c6f06b5..afb4e54 100644
--- a/java/kudu-client/src/test/java/org/kududb/client/BaseKuduTest.java
+++ b/java/kudu-client/src/test/java/org/kududb/client/BaseKuduTest.java
@@ -83,7 +83,9 @@ public class BaseKuduTest {
     masterHostPorts = miniCluster.getMasterHostPorts();
 
     LOG.info("Creating new Kudu client...");
-    client = new AsyncKuduClient.AsyncKuduClientBuilder(masterAddresses).build();
+    client = new AsyncKuduClient.AsyncKuduClientBuilder(masterAddresses)
+                                .defaultAdminOperationTimeoutMs(DEFAULT_SLEEP)
+                                .build();
     syncClient = new KuduClient(client);
     LOG.info("Waiting for tablet servers...");
     if (!miniCluster.waitForTabletServers(NUM_TABLET_SERVERS)) {

http://git-wip-us.apache.org/repos/asf/incubator-kudu/blob/830c3e4f/java/kudu-client/src/test/resources/log4j.properties
----------------------------------------------------------------------
diff --git a/java/kudu-client/src/test/resources/log4j.properties b/java/kudu-client/src/test/resources/log4j.properties
index 2d7619c..2d8d287 100644
--- a/java/kudu-client/src/test/resources/log4j.properties
+++ b/java/kudu-client/src/test/resources/log4j.properties
@@ -16,8 +16,8 @@
 # under the License.
 
 log4j.rootLogger = DEBUG, out
+log4j.logger.org.kududb = DEBUG, out
+
 log4j.appender.out = org.apache.log4j.ConsoleAppender
 log4j.appender.out.layout = org.apache.log4j.PatternLayout
 log4j.appender.out.layout.ConversionPattern = %d (%t) [%p - %l] %m%n
-
-log4j.logger.org.kududb = DEBUG