-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/44436/
-----------------------------------------------------------
(Updated March 17, 2016, 2:17 a.m.)
Review request for lens.
Changes
-------
1. Fixed Review comments .
2. Moved decidePriority() to AbstractQueryContext
3. Moved call to decidePriority in QueryExecutionServiceImpl to
rewriteAndSelect()
Bugs: lens-919
https://issues.apache.org/jira/browse/lens-919
Repository: lens
Description
-------
As of now Query Priority is not persisted. The query details API shows query
priority only until query is in memory. Once purged this info is not available .
Diffs (updated)
-----
lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java
f422543
lens-server-api/src/main/java/org/apache/lens/server/api/driver/AbstractLensDriver.java
d447417
lens-server-api/src/main/java/org/apache/lens/server/api/driver/LensDriver.java
ed97673
lens-server-api/src/main/java/org/apache/lens/server/api/query/AbstractQueryContext.java
62ed293
lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java
a57a6e4
lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java
96846c1
lens-server/src/main/java/org/apache/lens/server/query/LensServerDAO.java
d8e654d
lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java
49ab241
lens-server/src/test/java/org/apache/lens/server/query/TestLensDAO.java
760e306
lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java
699fa68
Diff: https://reviews.apache.org/r/44436/diff/
Testing
-------
updated test cases to check for query priority and tested them
-------------------------------------------------------
T E S T S
-------------------------------------------------------
Running org.apache.lens.server.query.TestQueryService
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 63.063 sec - in
org.apache.lens.server.query.TestQueryService
Results :
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0
Thanks,
Puneet Gupta
