You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lens.apache.org by pr...@apache.org on 2016/05/12 10:31:56 UTC

lens git commit: LENS-1041: Test case org.apache.lens.client.TestLensClient.testTimeout Failing intermittently

Repository: lens
Updated Branches:
  refs/heads/master 3d1b48c44 -> efa247f07


LENS-1041: Test case org.apache.lens.client.TestLensClient.testTimeout  Failing intermittently


Project: http://git-wip-us.apache.org/repos/asf/lens/repo
Commit: http://git-wip-us.apache.org/repos/asf/lens/commit/efa247f0
Tree: http://git-wip-us.apache.org/repos/asf/lens/tree/efa247f0
Diff: http://git-wip-us.apache.org/repos/asf/lens/diff/efa247f0

Branch: refs/heads/master
Commit: efa247f07c14760c1eb83226ba15eef8eabbc01b
Parents: 3d1b48c
Author: Puneet Gupta <pu...@gmail.com>
Authored: Thu May 12 16:00:55 2016 +0530
Committer: Rajat Khandelwal <ra...@gmail.com>
Committed: Thu May 12 16:00:55 2016 +0530

----------------------------------------------------------------------
 .../test/java/org/apache/lens/client/TestLensClient.java  | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lens/blob/efa247f0/lens-client/src/test/java/org/apache/lens/client/TestLensClient.java
----------------------------------------------------------------------
diff --git a/lens-client/src/test/java/org/apache/lens/client/TestLensClient.java b/lens-client/src/test/java/org/apache/lens/client/TestLensClient.java
index ff3b888..5cf7417 100644
--- a/lens-client/src/test/java/org/apache/lens/client/TestLensClient.java
+++ b/lens-client/src/test/java/org/apache/lens/client/TestLensClient.java
@@ -249,11 +249,12 @@ public class TestLensClient extends LensAllApplicationJerseyTest {
   public void testTimeout() throws LensAPIException {
     LensClientConfig config = createLensClientConfigWithServerUrl();
 
-    //Timeout Expected
-    config.setInt(LensClientConfig.READ_TIMEOUT_MILLIS, 200);
     LensClient lensClient = new LensClient(config);
     assertTrue(lensClient.setDatabase(TEST_DB));
     try {
+      // Setting very small timeout. Expecting timeouts after this
+      // Note: Timeout values can be changed even after LensClient has been created.
+      config.setInt(LensClientConfig.READ_TIMEOUT_MILLIS, 200);
       lensClient.executeQueryWithTimeout("cube select id,name from test_dim", "test1", 100000);
       fail("Read Timeout was expected");
     } catch (Exception e) {
@@ -264,11 +265,8 @@ public class TestLensClient extends LensAllApplicationJerseyTest {
         log.debug("Expected Exception", e);
       }
     }
-    lensClient.closeConnection();
 
-    //No Timeout Expected
-    lensClient = new LensClient(config);
-    assertTrue(lensClient.setDatabase(TEST_DB));
+    //Setting back default timeout. Not expecting timeouts after this
     config.setInt(LensClientConfig.READ_TIMEOUT_MILLIS, LensClientConfig.DEFAULT_READ_TIMEOUT_MILLIS);
     QueryHandleWithResultSet result = lensClient.executeQueryWithTimeout("cube select id,name from test_dim", "test2",
       100000);