-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43649/
-----------------------------------------------------------

(Updated Feb. 17, 2016, 3:32 p.m.)


Review request for lens and Amareshwari Sriramadasu.


Bugs: LENS-271
    https://issues.apache.org/jira/browse/LENS-271


Repository: lens


Description
-------

Changes for adding Druid driver in Lens


Diffs (updated)
-----

  lens-driver-druid/pom.xml PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/ASTTraverserForDruid.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/ColumnSchema.java 
PRE-CREATION 
  lens-driver-druid/src/main/java/com/apache/lens/driver/druid/DruidDriver.java 
PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/DruidDriverConfig.java
 PRE-CREATION 
  lens-driver-druid/src/main/java/com/apache/lens/driver/druid/DruidQuery.java 
PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/DruidQueryBuilder.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/client/DruidClient.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/client/DruidClientImpl.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/client/DruidResultSet.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/client/DruidResultSetTransformer.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/exceptions/DruidClientException.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/exceptions/DruidRewriteException.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/grammar/Aggregators.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/grammar/LogicalOperators.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/grammar/Predicates.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/grammar/having/HavingLogicalOperators.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/grammar/having/HavingPredicates.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/translator/DruidCriteriaVisitor.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/translator/DruidCriteriaVisitorFactory.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/translator/DruidHavingVisitor.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/translator/DruidHavingVisitorFactory.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/translator/DruidVisitor.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/translator/GroupByVisitor.java
 PRE-CREATION 
  
lens-driver-druid/src/main/java/com/apache/lens/driver/druid/translator/TopNVisitor.java
 PRE-CREATION 
  lens-driver-druid/src/main/resources/druiddriver-default.xml PRE-CREATION 
  
lens-driver-druid/src/test/java/org/apache/lens/driver/druid/DruidInitDriverTest.java
 PRE-CREATION 
  
lens-driver-druid/src/test/java/org/apache/lens/driver/druid/MockClientDruid.java
 PRE-CREATION 
  
lens-driver-druid/src/test/java/org/apache/lens/driver/druid/QueryTranslationTest.java
 PRE-CREATION 
  
lens-driver-druid/src/test/java/org/apache/lens/driver/druid/ResultSetTransformationTest.java
 PRE-CREATION 
  lens-driver-druid/src/test/resources/druiddriver-default.xml PRE-CREATION 
  lens-driver-druid/src/test/resources/hive-site.xml PRE-CREATION 
  lens-driver-druid/src/test/resources/invalid-queries.data PRE-CREATION 
  lens-driver-druid/src/test/resources/valid-queries.data PRE-CREATION 
  lens-driver-es/src/main/java/org/apache/lens/driver/es/ASTTraverserForES.java 
07b157e 
  lens-driver-es/src/main/java/org/apache/lens/driver/es/ESDriverConfig.java 
8f293f5 
  
lens-driver-es/src/main/java/org/apache/lens/driver/es/client/jest/JestResultSetTransformer.java
 38d91f9 
  
lens-driver-es/src/main/java/org/apache/lens/driver/es/exceptions/InvalidQueryException.java
 20634af 
  
lens-driver-es/src/main/java/org/apache/lens/driver/es/grammar/Aggregations.java
 f726fa5 
  
lens-driver-es/src/main/java/org/apache/lens/driver/es/grammar/LogicalOperators.java
 b9cf000 
  
lens-driver-es/src/main/java/org/apache/lens/driver/es/grammar/Predicates.java 
ec2af0f 
  
lens-driver-es/src/main/java/org/apache/lens/driver/es/translator/ASTCriteriaVisitor.java
 b429424 
  
lens-driver-es/src/main/java/org/apache/lens/driver/es/translator/ASTVisitor.java
 77e774f 
  
lens-driver-es/src/main/java/org/apache/lens/driver/es/translator/CriteriaVisitorFactory.java
 92ec10f 
  
lens-driver-es/src/main/java/org/apache/lens/driver/es/translator/ESVisitor.java
 441f6d6 
  
lens-driver-es/src/main/java/org/apache/lens/driver/es/translator/impl/ESAggregateVisitor.java
 e8f2cea 
  
lens-driver-es/src/main/java/org/apache/lens/driver/es/translator/impl/ESCriteriaVisitor.java
 d1bf2a4 
  
lens-driver-es/src/main/java/org/apache/lens/driver/es/translator/impl/ESCriteriaVisitorFactory.java
 04b773d 
  
lens-driver-es/src/main/java/org/apache/lens/driver/es/translator/impl/ESTermVisitor.java
 49aa0d4 
  
lens-driver-es/src/test/java/org/apache/lens/driver/es/ResultSetTransformationTest.java
 0b78639 
  lens-examples/src/main/resources/cube-queries.sql 9f4a353 
  lens-examples/src/main/resources/dimension-queries.sql a5f51d9 
  
lens-server-api/src/main/java/org/apache/lens/server/api/driver/lib/ASTCriteriaVisitor.java
 PRE-CREATION 
  
lens-server-api/src/main/java/org/apache/lens/server/api/driver/lib/ASTVisitor.java
 PRE-CREATION 
  
lens-server-api/src/main/java/org/apache/lens/server/api/driver/lib/CriteriaVisitorFactory.java
 PRE-CREATION 
  
lens-server-api/src/main/java/org/apache/lens/server/api/driver/lib/exception/InvalidQueryException.java
 PRE-CREATION 
  pom.xml b3afa59 
  src/site/apt/admin/druiddriver-config.apt PRE-CREATION 

Diff: https://reviews.apache.org/r/43649/diff/


Testing
-------

[INFO] Reactor Summary:
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [1.883s]
[INFO] Lens .............................................. SUCCESS [3.355s]
[INFO] Lens API .......................................... SUCCESS [25.446s]
[INFO] Lens API for server and extensions ................ SUCCESS [19.944s]
[INFO] Lens Cube ......................................... SUCCESS [10:55.222s]
[INFO] Lens DB storage ................................... SUCCESS [20.357s]
[INFO] Lens Query Library ................................ SUCCESS [15.574s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:49.465s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [37.598s]
[INFO] Lens Elastic Search Driver ........................ SUCCESS [16.879s]
[INFO] Lens Driver for Druid ............................. SUCCESS [27.236s]
[INFO] Lens Server ....................................... SUCCESS [10:03.541s]
[INFO] Lens client ....................................... SUCCESS [35.725s]
[INFO] Lens CLI .......................................... SUCCESS [50.380s]
[INFO] Lens Examples ..................................... SUCCESS [8.416s]
[INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [0.701s]
[INFO] Lens Distribution ................................. SUCCESS [8.340s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:20.841s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [1.998s]
[INFO] Lens Regression ................................... SUCCESS [14.291s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 29:58.174s
[INFO] Finished at: Wed Feb 17 11:13:45 UTC 2016
[INFO] Final Memory: 199M/1276M
[INFO] ------------------------------------------------------------------------


Thanks,

Rajitha R

Reply via email to