----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51548/#review147545 -----------------------------------------------------------
lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java (line 214) <https://reviews.apache.org/r/51548/#comment214729> Need the reverse implementation too for the sake of completeness. The function `toQueryContext` should be able to take driver query from finished query and set inside query context. This will allow the driver query to be passed to the query details api too. - Rajat Khandelwal On Aug. 31, 2016, 5:20 p.m., Raghavendra Singh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51548/ > ----------------------------------------------------------- > > (Updated Aug. 31, 2016, 5:20 p.m.) > > > Review request for lens. > > > Bugs: LENS-1304 > https://issues.apache.org/jira/browse/LENS-1304 > > > Repository: lens > > > Description > ------- > > Make driver query available in the finished_queries stats for deeper analytics > > > Diffs > ----- > > > lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java > 1d56757 > lens-server/src/main/java/org/apache/lens/server/query/LensServerDAO.java > 47159b5 > lens-server/src/test/java/org/apache/lens/server/query/TestLensDAO.java > 74d9a1f > > Diff: https://reviews.apache.org/r/51548/diff/ > > > Testing > ------- > > Test added in TestLensDAO.java > > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Checkstyle Rules .............................. SUCCESS [ 12.621 > s] > [INFO] Lens ............................................... SUCCESS [ 14.930 > s] > [INFO] Lens API ........................................... SUCCESS [ 44.131 > s] > [INFO] Lens API for server and extensions ................. SUCCESS [ 46.240 > s] > [INFO] Lens Cube .......................................... SUCCESS [21:11 > min] > [INFO] Lens DB storage .................................... SUCCESS [ 32.606 > s] > [INFO] Lens Query Library ................................. SUCCESS [ 28.387 > s] > [INFO] Lens Hive Driver ................................... SUCCESS [02:47 > min] > [INFO] Lens Driver for JDBC ............................... SUCCESS [ 56.947 > s] > [INFO] Lens Elastic Search Driver ......................... SUCCESS [ 33.325 > s] > [INFO] Lens Server ........................................ SUCCESS [22:44 > min] > [INFO] Lens client ........................................ SUCCESS [02:10 > min] > [INFO] Lens CLI ........................................... SUCCESS [02:07 > min] > [INFO] Lens Examples ...................................... SUCCESS [ 16.364 > s] > [INFO] Lens Ship Jars to Distributed Cache ................ SUCCESS [ 2.979 > s] > [INFO] Lens Distribution .................................. SUCCESS [ 38.824 > s] > [INFO] Lens ML Lib ........................................ SUCCESS [02:40 > min] > [INFO] Lens ML Ext Distribution ........................... SUCCESS [ 22.394 > s] > [INFO] Lens Regression .................................... SUCCESS [ 26.432 > s] > [INFO] Lens UI ............................................ SUCCESS [ 21.591 > s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 01:00 h > [INFO] Finished at: 2016-08-30T19:25:13+05:30 > [INFO] Final Memory: 199M/1058M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Raghavendra Singh > >
