-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32393/
-----------------------------------------------------------
(Updated March 24, 2015, 9:36 a.m.)
Review request for lens, Jaideep dhok and Rajat Khandelwal.
Changes
-------
Review comments incorporated
Bugs: LENS-323
https://issues.apache.org/jira/browse/LENS-323
Repository: lens
Description (updated)
-------
WIP patch for early comments.
Changes include:
- Generating rewriter plan with cube queries involved
- Populating FactPartitions queried in RewriterPlan
- priority getting set only for cube queries; native tables queries are left
with defaults
- TestHiveDriver.testPriority updated
- Fix other test failures
- Clear transient state in QueryContext after launch.
Still pending
- Add unit tests for rewriter plan construction
Diffs (updated)
-----
lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateDim.java 3ac207e
lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateFact.java 66c3b25
lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTable.java
5eaf7ec
lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryContext.java
df5a212
lens-cube/src/main/java/org/apache/lens/cube/parse/StorageTableResolver.java
c29d88f
lens-cube/src/main/java/org/apache/lens/driver/cube/RewriteUtil.java 8ed4070
lens-cube/src/main/java/org/apache/lens/driver/cube/RewriterPlan.java
PRE-CREATION
lens-driver-hive/pom.xml 3b8e73a
lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java
9e3c723
lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveQueryPlan.java
64a6bba
lens-driver-hive/src/main/java/org/apache/lens/driver/hive/priority/DurationBasedQueryPriorityDecider.java
4f471cb
lens-driver-hive/src/test/java/org/apache/lens/driver/hive/TestHiveDriver.java
8a5839b
lens-driver-hive/src/test/resources/priority_tests.data 230df95
lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryPlan.java
38ebc55
lens-server-api/src/main/java/org/apache/lens/server/api/query/AbstractQueryContext.java
5233569
lens-server-api/src/main/java/org/apache/lens/server/api/query/DriverSelectorQueryContext.java
fef834f
lens-server-api/src/test/java/org/apache/lens/server/api/driver/MockDriver.java
67c73b8
lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java
390071c
Diff: https://reviews.apache.org/r/32393/diff/
Testing
-------
Thanks,
Amareshwari Sriramadasu
