----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63665/#review190922 -----------------------------------------------------------
lens-cube/src/main/java/org/apache/lens/cube/query/cost/StaticCostCalculator.java Lines 36 (patched) <https://reviews.apache.org/r/63665/#comment268491> Since this is a static cost and cost type per driver, We should try not to evaluate it each time from conf. lens-cube/src/test/java/org/apache/lens/cube/query/cost/TestFactPartitionBasedQueryCostCalculator.java Lines 54 (patched) <https://reviews.apache.org/r/63665/#comment268490> Shoudl be call CostRangeQueryTypeDecider as RangeBasedQueryCostTypeDecider lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java Lines 99 (patched) <https://reviews.apache.org/r/63665/#comment268492> Shoul we have default as "LOW,0.0,HIGH" ? lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java Lines 143 (patched) <https://reviews.apache.org/r/63665/#comment268493> Who should create the instance of QueryCostTypeDecider . Driver or QueryCostCalculator ? lens-driver-jdbc/src/main/resources/jdbcdriver-default.xml Lines 293 (patched) <https://reviews.apache.org/r/63665/#comment268494> Should we have default as LOW,0.0,HIGH ? lens-server-api/src/main/java/org/apache/lens/server/api/query/cost/CostRangeQueryTypeDecider.java Lines 37 (patched) <https://reviews.apache.org/r/63665/#comment268495> The constructor can take the config string "LOW,0.0,HIGH" and costToQueryTypeRangeMap can be internal to the implementation of this class. lens-server-api/src/main/java/org/apache/lens/server/api/query/cost/CostToQueryTypeRangeConf.java Lines 28 (patched) <https://reviews.apache.org/r/63665/#comment268496> Should we rename this class also? QueryCostTypeRangeConf lens-server-api/src/main/java/org/apache/lens/server/api/query/cost/QueryCostTypeDecider.java Lines 29 (patched) <https://reviews.apache.org/r/63665/#comment268497> update comments - Puneet Gupta On Nov. 14, 2017, 6:25 a.m., Rajitha R wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63665/ > ----------------------------------------------------------- > > (Updated Nov. 14, 2017, 6:25 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-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/CostRangeQueryTypeDecider.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/cost/CostToQueryTypeRangeConf.java > PRE-CREATION > > 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/QueryCostTOBuilder.java > 81f1fd1 > > 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/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/MockQueryCostCalculator.java > fb4085e > > 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/5/ > > > Testing > ------- > > > Thanks, > > Rajitha R > >
