----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52337/ -----------------------------------------------------------
Review request for lens. Bugs: lens-1324 https://issues.apache.org/jira/browse/lens-1324 Repository: lens Description ------- Currently we have two hooks 1. preLaunch 2. postDriverSelection Need to add some hooks that can hook into rewrite and estimate flow also. 1.preRewrite, postRewrite 2.preEstimate,postEstimate NoOpDriverQueryHook will have default empty implementation for these new methods . Also added void setDriver(LensDriver driver) in Driver Hook to pass the driver information. Diffs ----- lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java 84d9933da506e3f1bff2d3e0ea403ed1edf0fe35 lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/JDBCDriver.java 3bf5e8f7dc9d032e46d06c3245048096acb4572e lens-server-api/src/main/java/org/apache/lens/server/api/driver/AbstractLensDriver.java 03079e2dc353fa028f3c5d5a03a8ae2dbf728004 lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryHook.java da125bd920aea82dc910fad92290d07fcc2a2387 lens-server-api/src/main/java/org/apache/lens/server/api/driver/NoOpDriverQueryHook.java 8cd03cb11ae88413973a9fc384fd828014b1aa0c lens-server-api/src/main/java/org/apache/lens/server/api/query/cost/FactPartitionBasedQueryCost.java 792508b14b9228ff20437d03f78e38165d920798 lens-server-api/src/test/java/org/apache/lens/server/api/driver/MockDriver.java 168b3cc8a2e32e4fddb66f480af38e77adef62c2 lens-server-api/src/test/java/org/apache/lens/server/api/user/MockDriverQueryHook.java 0c4a3f05ce752091d2187badf2054c3926c7c9f6 lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 0bb106ed77306094710c126fc652d5c56a908d85 lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java f755508529f6745caadd8eee0f63e81e1f14ed75 Diff: https://reviews.apache.org/r/52337/diff/ Testing ------- ------------------------------------------------------- T E S T S ------------------------------------------------------- Running org.apache.lens.server.query.TestQueryService Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 84.906 sec - in org.apache.lens.server.query.TestQueryService Results : Tests run: 2, Failures: 0, Errors: 0, Skipped: 0 [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 04:41 min [INFO] Finished at: 2016-09-28T11:47:07+05:30 [INFO] Final Memory: 133M/1482M [INFO] ------------------------------------------------------------------------ Thanks, Puneet Gupta