----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63665/#review191044 -----------------------------------------------------------
lens-cube/src/main/java/org/apache/lens/cube/query/cost/StaticCostCalculator.java Lines 43 (patched) <https://reviews.apache.org/r/63665/#comment268632> Can we push this to constructor? lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java Line 343 (original), 349 (patched) <https://reviews.apache.org/r/63665/#comment268633> Can we have a init() method that can do this and have a default constructor ? lens-driver-hive/src/main/resources/hivedriver-default.xml Lines 171 (patched) <https://reviews.apache.org/r/63665/#comment268639> update description. lens-driver-hive/src/main/resources/hivedriver-default.xml Lines 176 (patched) <https://reviews.apache.org/r/63665/#comment268634> can we have it same as default in code "LOW,0.0,HIGH" lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/JDBCDriver.java Lines 395 (patched) <https://reviews.apache.org/r/63665/#comment268635> can we have common code in AbstractDriver ? lens-driver-jdbc/src/main/resources/jdbcdriver-default.xml Lines 282 (patched) <https://reviews.apache.org/r/63665/#comment268637> update description. lens-driver-jdbc/src/main/resources/jdbcdriver-default.xml Lines 288 (patched) <https://reviews.apache.org/r/63665/#comment268638> update description lens-driver-jdbc/src/main/resources/jdbcdriver-default.xml Lines 293 (patched) <https://reviews.apache.org/r/63665/#comment268636> can we have it same as default in code - Puneet Gupta On Nov. 15, 2017, 10:32 a.m., Rajitha R wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63665/ > ----------------------------------------------------------- > > (Updated Nov. 15, 2017, 10:32 a.m.) > > > 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/query/cost/FactPartitionBasedQueryCostCalculator.java > 3c157ee > > lens-cube/src/main/java/org/apache/lens/cube/query/cost/StaticCostCalculator.java > PRE-CREATION > > lens-cube/src/test/java/org/apache/lens/cube/query/cost/TestFactPartitionBasedQueryCostCalculator.java > b5e73fd > lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java > 0b4db9a > lens-driver-hive/src/main/resources/hivedriver-default.xml a13d3b0 > > lens-driver-hive/src/test/java/org/apache/lens/driver/hive/TestHiveDriver.java > 1261409 > 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/driver/DriverQueryPlan.java > 2036dce > > lens-server-api/src/main/java/org/apache/lens/server/api/query/cost/FactPartitionBasedQueryCost.java > eba8f0d > > lens-server-api/src/main/java/org/apache/lens/server/api/query/cost/QueryCost.java > 9d7320a > > lens-server-api/src/main/java/org/apache/lens/server/api/query/cost/QueryCostCalculator.java > 8c71c00 > > lens-server-api/src/main/java/org/apache/lens/server/api/query/cost/QueryCostTypeDecider.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/cost/QueryCostTypeRangeConf.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/cost/RangeBasedQueryCostTypeDecider.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/cost/StaticQueryCost.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/priority/RangeConf.java > 43cc53c > > lens-server-api/src/test/java/org/apache/lens/server/api/query/cost/TestFactPartitionBasedQueryCost.java > e047c91 > src/site/apt/admin/hivedriver-config.apt a9f020d > src/site/apt/admin/jdbcdriver-config.apt eb56927 > > > Diff: https://reviews.apache.org/r/63665/diff/13/ > > > Testing > ------- > > > Thanks, > > Rajitha R > >
