----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46552/#review130802 -----------------------------------------------------------
[INFO] ------------------------------------------------------------------------ [INFO] Reactor Summary: [INFO] [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.500s] [INFO] Lens .............................................. SUCCESS [2.907s] [INFO] Lens API .......................................... SUCCESS [20.627s] [INFO] Lens API for server and extensions ................ SUCCESS [14.890s] [INFO] Lens Cube ......................................... SUCCESS [7:05.009s] [INFO] Lens DB storage ................................... SUCCESS [14.722s] [INFO] Lens Query Library ................................ SUCCESS [12.032s] [INFO] Lens Hive Driver .................................. SUCCESS [2:24.438s] [INFO] Lens Driver for JDBC .............................. SUCCESS [31.237s] [INFO] Lens Elastic Search Driver ........................ SUCCESS [13.009s] [INFO] Lens Server ....................................... SUCCESS [13:49.378s] [INFO] Lens client ....................................... SUCCESS [33.915s] [INFO] Lens CLI .......................................... SUCCESS [2:10.859s] [INFO] Lens Examples ..................................... SUCCESS [7.950s] [INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [0.643s] [INFO] Lens Distribution ................................. SUCCESS [9.000s] [INFO] Lens ML Lib ....................................... SUCCESS [1:09.007s] [INFO] Lens ML Ext Distribution .......................... SUCCESS [1.403s] [INFO] Lens Regression ................................... SUCCESS [10.670s] [INFO] Lens UI ........................................... SUCCESS [45.606s] [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 30:19.662s [INFO] Finished at: Wed Apr 27 15:42:54 UTC 2016 [INFO] Final Memory: 173M/2840M [INFO] ------------------------------------------------------------------------ [JDK Parameter]: Restored job JDK - Puneet Gupta On April 27, 2016, 3:44 p.m., Puneet Gupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46552/ > ----------------------------------------------------------- > > (Updated April 27, 2016, 3:44 p.m.) > > > Review request for lens. > > > Bugs: lens-1013 > https://issues.apache.org/jira/browse/lens-1013 > > > Repository: lens > > > Description > ------- > > as of now DriverQueryHook has a preLaunch() method which is called just > before launching the query . > > We should also have an option to interact with hook on postSelect() which > will be called once query has been accepted by lens server and appropriate > driver has been selected for it. > > > Also as of now, the preLaunch() is called by individual driver. This should > be moved to server and drivers should not manage this operation. > > > Diffs > ----- > > lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java > 6eec3f7 > > lens-driver-hive/src/test/java/org/apache/lens/driver/hive/TestHiveDriver.java > 3f77b0c > > lens-driver-hive/src/test/java/org/apache/lens/driver/hive/TestRemoteHiveDriver.java > 4f18c24 > lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/JDBCDriver.java > eef4464 > > lens-driver-jdbc/src/test/java/org/apache/lens/driver/jdbc/TestJdbcDriver.java > 81a9552 > > lens-server-api/src/main/java/org/apache/lens/server/api/driver/AbstractLensDriver.java > 883ad9d > > lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryHook.java > ecac6be > > lens-server-api/src/main/java/org/apache/lens/server/api/driver/LensDriver.java > 69295d9 > > lens-server-api/src/main/java/org/apache/lens/server/api/driver/NoOpDriverQueryHook.java > a5475e4 > > lens-server-api/src/test/java/org/apache/lens/server/api/user/MockDriverQueryHook.java > 2505ef4 > > lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java > c72a890 > lens-server/src/test/java/org/apache/lens/server/LensJerseyTest.java > fb757e4 > lens-server/src/test/java/org/apache/lens/server/TestServerRestart.java > 0f55d9e > > lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java > 6bf077d > lens-server/src/test/resources/drivers/hive/hive1/hivedriver-site.xml > 6362473 > lens-server/src/test/resources/drivers/hive/hive2/hivedriver-site.xml > d5be96e > > Diff: https://reviews.apache.org/r/46552/diff/ > > > Testing > ------- > > mvn test -Dtest=org.apache.lens.server.query.TestQueryService#testExecuteAsync > ------------------------------------------------------- > T E S T S > ------------------------------------------------------- > Running org.apache.lens.server.query.TestQueryService > Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 116.959 sec - > in org.apache.lens.server.query.TestQueryService > > Results : > > Tests run: 2, Failures: 0, Errors: 0, Skipped: 0 > > Logs from MockDriverQueryHook > ----- > > 22 Apr 2016 02:32:11 f5f0df2e-b8c5-4b10-b9a9-d714f3be6caf > [grizzly-http-server-3] INFO > org.apache.lens.server.api.driver.NoOpDriverQueryHook - Post select for foo, > user query: select ID, IDSTR from TEST_TABLE, driver hive/hive1, driver > query: select ID, IDSTR from TEST_TABLE > > > 22 Apr 2016 02:32:43 4d3c9dc6-2abf-432a-aa94-acb937c1b457 [QuerySubmitter] > INFO org.apache.lens.server.api.driver.NoOpDriverQueryHook - Pre launch for > foo, user query: select ID, IDSTR from TEST_TABLE, driver hive/hive1, driver > query: select ID, IDSTR from TEST_TABLE > > > Pending > ------ > Will try to update server restart test case and MockQueryDriverHook to use > AbstractQueryContext#updateConf(Map) > > > Thanks, > > Puneet Gupta > >
