> On May 2, 2016, 11:17 a.m., Amareshwari Sriramadasu wrote: > > lens-client/src/main/java/org/apache/lens/client/LensStatement.java, line > > 317 > > <https://reviews.apache.org/r/46767/diff/2/?file=1368323#file1368323line317> > > > > Shall we log exception as well?
Actually getQuery(handle) is logging the exception before throwing it. So not logging the stacktrace here . > On May 2, 2016, 11:17 a.m., Amareshwari Sriramadasu wrote: > > lens-client/src/test/java/org/apache/lens/server/MockQueryExecutionServiceImpl.java, > > lines 34-35 > > <https://reviews.apache.org/r/46767/diff/2/?file=1368326#file1368326line34> > > > > Can we reduce these further and the timeouts also, sothat test time > > reduces? With 5 seconds, number of retries = 2 ( max 3) 5 secs = lensClient.getStatement().waitForQueryToComplete(handle, false); // no retry case 4 secs = lensClient.getStatement().waitForQueryToComplete(handle); // try 1 3 secs = lensClient.getStatement().waitForQueryToComplete(handle); // retry 1 2 secs = lensClient.getStatement().waitForQueryToComplete(handle); // retry 2 1 secs = lensClient.getStatement().waitForQueryToComplete(handle); // retry 3(Sometimes this retry is hit) Total time is testTimeout = 5.6 secs testWaitForQueryToCompleteWithAndWithoutRetryOnTimeOut = 13 secs - Puneet ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46767/#review131288 ----------------------------------------------------------- On May 2, 2016, 10:34 a.m., Puneet Gupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46767/ > ----------------------------------------------------------- > > (Updated May 2, 2016, 10:34 a.m.) > > > Review request for lens. > > > Bugs: lens-383 > https://issues.apache.org/jira/browse/lens-383 > > > Repository: lens > > > Description > ------- > > To handle cases where server is not responding or slow in response, we need > to put timeouts on api calls in lens client. > > Note: > This patch hanlde changes for lens-client users only (lens-regression and > lens-ml-lib are also creating clients for limited functionality which is not > addressed here) > > > Diffs > ----- > > lens-client/src/main/java/org/apache/lens/client/LensClientConfig.java > 6a35a5e > lens-client/src/main/java/org/apache/lens/client/LensConnection.java > 0c2557c > lens-client/src/main/java/org/apache/lens/client/LensStatement.java f06bcd1 > lens-client/src/main/resources/lens-client-default.xml 5a578a7 > lens-client/src/test/java/org/apache/lens/client/TestLensClient.java > 6e638f1 > > lens-client/src/test/java/org/apache/lens/server/MockQueryExecutionServiceImpl.java > PRE-CREATION > src/site/apt/admin/config.apt b5853bf > src/site/apt/user/client-config.apt 4ed41b4 > > Diff: https://reviews.apache.org/r/46767/diff/ > > > Testing > ------- > > [INFO] > > [INFO] > ------------------------------------------------------------------------ > [INFO] Building Lens client 2.6.0-beta-SNAPSHOT > [INFO] > ------------------------------------------------------------------------ > [INFO] > [INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ lens-client --- > [INFO] > [INFO] --- maven-remote-resources-plugin:1.5:process (default) @ lens-client > --- > [INFO] > [INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ > lens-client --- > [INFO] Using 'UTF-8' encoding to copy filtered resources. > [INFO] skip non existing resourceDirectory > /d0/jenkins/workspace/Platform_Common_Job/lens-client/src/deb/control > [INFO] Copying 1 resource > [INFO] Copying 3 resources > [INFO] > [INFO] --- maven-compiler-plugin:3.1:compile (default-compile) @ lens-client > --- > [INFO] Changes detected - recompiling the module! > [INFO] Compiling 34 source files to > /d0/jenkins/workspace/Platform_Common_Job/lens-client/target/classes > [WARNING] > /d0/jenkins/workspace/Platform_Common_Job/lens-client/src/main/java/org/apache/lens/client/jdbc/LensJdbcResultSet.java: > > /d0/jenkins/workspace/Platform_Common_Job/lens-client/src/main/java/org/apache/lens/client/jdbc/LensJdbcResultSet.java > uses or overrides a deprecated API. > [WARNING] > /d0/jenkins/workspace/Platform_Common_Job/lens-client/src/main/java/org/apache/lens/client/jdbc/LensJdbcResultSet.java: > Recompile with -Xlint:deprecation for details. > [INFO] > [INFO] --- maven-dependency-plugin:2.8:build-classpath (build-classpath) @ > lens-client --- > [INFO] Wrote classpath file > '/d0/jenkins/workspace/Platform_Common_Job/lens-client/target/classpath'. > [INFO] > [INFO] --- maven-resources-plugin:2.6:testResources (default-testResources) @ > lens-client --- > [INFO] Using 'UTF-8' encoding to copy filtered resources. > [INFO] Copying 9 resources > [INFO] Copying 3 resources > [INFO] > [INFO] --- maven-compiler-plugin:3.1:testCompile (default-testCompile) @ > lens-client --- > [INFO] Changes detected - recompiling the module! > [INFO] Compiling 5 source files to > /d0/jenkins/workspace/Platform_Common_Job/lens-client/target/test-classes > [INFO] > [INFO] --- maven-surefire-plugin:2.15:test (default-test) @ lens-client --- > [INFO] Surefire report directory: > /d0/jenkins/workspace/Platform_Common_Job/lens-client/target/surefire-reports > > ------------------------------------------------------- > T E S T S > ------------------------------------------------------- > Java HotSpot(TM) 64-Bit Server VM warning: ignoring option PermSize=256m; > support was removed in 8.0 > Java HotSpot(TM) 64-Bit Server VM warning: ignoring option MaxPermSize=256m; > support was removed in 8.0 > Running TestSuite > Tests run: 25, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 84.336 sec - > in TestSuite > > Results : > > Tests run: 25, Failures: 0, Errors: 0, Skipped: 0 > > > Thanks, > > Puneet Gupta > >
