----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30159/#review69153 -----------------------------------------------------------
Ship it! Ship It! - Jaideep dhok On Jan. 22, 2015, 10:06 a.m., Amareshwari Sriramadasu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30159/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2015, 10:06 a.m.) > > > Review request for lens and Jaideep dhok. > > > Bugs: LENS-223 > https://issues.apache.org/jira/browse/LENS-223 > > > Repository: lens > > > Description > ------- > > Changes include : > * Makes the lens-server tests run with two drivers > * Fixes createResourceQuery to properly create query context > * Cleaned up QueryContext constructors - Now there are only two constructors > - one passing query, other passing PreparedQueryContext. > * Added utility createContextWithSingleDriver for creating context with > single driver - used in resource query, explain query and tests > > > Diffs > ----- > > lens-cli/pom.xml 97fa433 > lens-client/pom.xml fa771e4 > lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java > 64bc281 > > lens-driver-hive/src/test/java/org/apache/lens/driver/hive/TestHiveDriver.java > c63a256 > lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/JDBCDriver.java > 1b4982a > > lens-driver-jdbc/src/test/java/org/apache/lens/driver/jdbc/TestJDBCFinal.java > 7534d89 > > lens-driver-jdbc/src/test/java/org/apache/lens/driver/jdbc/TestJdbcDriver.java > e1988d4 > > lens-query-lib/src/test/java/org/apache/lens/lib/query/TestAbstractFileFormatter.java > bc480ec > > lens-server-api/src/main/java/org/apache/lens/server/api/query/FinishedLensQuery.java > 0971f86 > > lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java > abaa5f0 > lens-server/pom.xml 10accd9 > > lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java > c95a948 > > lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java > f964037 > lens-server/src/test/resources/jdbcdriver-site.xml PRE-CREATION > lens-server/src/test/resources/lens-site.xml ebf4ddb > > Diff: https://reviews.apache.org/r/30159/diff/ > > > Testing > ------- > > TestSessionResource passes with changes, was failing without fix when running > with two drivers. All tests pass with latest > > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.492s] > [INFO] Lens .............................................. SUCCESS [1.853s] > [INFO] Lens API .......................................... SUCCESS [6.431s] > [INFO] Lens API for server and extensions ................ SUCCESS [7.133s] > [INFO] Lens Cube ......................................... SUCCESS [7:14.549s] > [INFO] Lens DB storage ................................... SUCCESS [11.782s] > [INFO] Lens Query Library ................................ SUCCESS [5.672s] > [INFO] Lens Hive Driver .................................. SUCCESS [3:17.076s] > [INFO] Lens Driver for Cloudera Impala ................... SUCCESS [3.583s] > [INFO] Lens Driver for JDBC .............................. SUCCESS [29.501s] > [INFO] Lens Server ....................................... SUCCESS [5:14.872s] > [INFO] Lens client ....................................... SUCCESS [27.921s] > [INFO] Lens CLI .......................................... SUCCESS [2:00.760s] > [INFO] Lens Examples ..................................... SUCCESS [1.156s] > [INFO] Lens Distribution ................................. SUCCESS [4.223s] > [INFO] Lens Client Distribution .......................... SUCCESS [8.206s] > [INFO] Lens ML Lib ....................................... SUCCESS [52.684s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 20:30.962s > [INFO] Finished at: Thu Jan 22 10:06:29 UTC 2015 > [INFO] Final Memory: 110M/1284M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Amareshwari Sriramadasu > >
