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

Review request for lens and Puneet Gupta.


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


Repository: lens


Description
-------

Jdbc driver cost has been hardcoded to zero currently in Lens code. This makes 
it difficult to add more jdbc drivers which have different costs. Hence there 
is a need for having a driver level implementation for calculating cost.


Diffs
-----

  lens-api/src/main/java/org/apache/lens/api/query/QueryCostType.java 37eda42 
  
lens-cube/src/main/java/org/apache/lens/cube/metadata/CubeVirtualFactTable.java 
8b55985 
  lens-cube/src/main/java/org/apache/lens/cube/parse/StorageCandidate.java 
fd07bf2 
  
lens-cube/src/main/java/org/apache/lens/cube/query/cost/StaticCostCalculator.java
 PRE-CREATION 
  lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/JDBCDriver.java 
528f857 
  
lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/JDBCDriverConfConstants.java
 51abc96 
  lens-driver-jdbc/src/main/resources/jdbcdriver-default.xml 5545fac 
  
lens-driver-jdbc/src/test/java/org/apache/lens/driver/jdbc/TestJdbcDriver.java 
446e759 
  
lens-server-api/src/main/java/org/apache/lens/server/api/LensConfConstants.java 
7fd487c 
  
lens-server-api/src/main/java/org/apache/lens/server/api/query/cost/StaticQueryCost.java
 PRE-CREATION 
  
lens-server-api/src/test/java/org/apache/lens/server/api/query/cost/TestFactPartitionBasedQueryCost.java
 e047c91 
  src/site/apt/admin/jdbcdriver-config.apt eb56927 


Diff: https://reviews.apache.org/r/63665/diff/1/


Testing
-------


Thanks,

Rajitha R

Reply via email to