-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47159/
-----------------------------------------------------------
(Updated May 10, 2016, 11:06 a.m.)
Review request for lens.
Changes
-------
- Made cancel on timeout false for lens client ( CLI to be specific)
- Fixed find bug in QueryExecutionServiceImpl.java
- Fixde failing test case
testExecuteWithTimeoutAndPreFetechAndServerPersistence()
Bugs: lens-1032
https://issues.apache.org/jira/browse/lens-1032
Repository: lens
Description
-------
Right now, on execute with timeout api, if the query times out, then query will
be still running for people to pull reports.
We should provide option to kill the query upon timeout for users who are not
interested in result beyond timeout.
Chnages
------
Added property lens.query.cancel.on.timeout (default = true )
Added queryCancellationPool to Query Service to cancel queries asynchronously
Updated Timeout logic in QueryExecutionServiceImpl#executeTimeoutInternal
HiveDriver : updated check interval for completion listener . Minumum interval
is 5 secs now.
Diffs (updated)
-----
lens-api/src/main/java/org/apache/lens/api/query/QueryStatus.java 67d1e79
lens-client/src/main/resources/lens-client-default.xml 96506ac
lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java
1463cc2
lens-server-api/src/main/java/org/apache/lens/server/api/LensConfConstants.java
7cf6449
lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java
b12943e
lens-server/src/main/java/org/apache/lens/server/query/QueryServiceResource.java
a043550
lens-server/src/main/resources/lenssession-default.xml ce296cc
lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java
62c0280
src/site/apt/admin/session-config.apt 206d6a4
Diff: https://reviews.apache.org/r/47159/diff/
Testing
-------
org.apache.lens.server.query.TestQueryService#testAutoCancelOnTimeOut : Pass
Thanks,
Puneet Gupta
