----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34286/ -----------------------------------------------------------
(Updated May 21, 2015, 7:08 p.m.) Review request for lens. Changes ------- Build results without testcases: Blocked by: https://issues.apache.org/jira/browse/LENS-573 [INFO] ------------------------------------------------------------------------ [INFO] Reactor Summary: [INFO] [INFO] Lens Checkstyle Rules .............................. SUCCESS [ 2.437 s] [INFO] Lens ............................................... SUCCESS [ 2.977 s] [INFO] Lens API ........................................... SUCCESS [ 19.279 s] [INFO] Lens API for server and extensions ................. SUCCESS [ 20.546 s] [INFO] Lens Cube .......................................... SUCCESS [ 39.771 s] [INFO] Lens DB storage .................................... SUCCESS [ 10.072 s] [INFO] Lens Query Library ................................. SUCCESS [ 12.768 s] [INFO] Lens Hive Driver ................................... SUCCESS [ 14.800 s] [INFO] Lens Driver for JDBC ............................... SUCCESS [ 17.623 s] [INFO] Lens Server ........................................ SUCCESS [ 41.252 s] [INFO] Lens client ........................................ SUCCESS [ 17.651 s] [INFO] Lens CLI ........................................... SUCCESS [ 15.301 s] [INFO] Lens Examples ...................................... SUCCESS [ 13.990 s] [INFO] Lens Distribution .................................. SUCCESS [ 7.776 s] [INFO] Lens ML Lib ........................................ SUCCESS [ 19.744 s] [INFO] Lens ML Ext Distribution ........................... SUCCESS [ 2.349 s] [INFO] Lens Regression .................................... SUCCESS [ 7.476 s] [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 04:26 min [INFO] Finished at: 2015-05-22T00:34:28+05:30 [INFO] Final Memory: 146M/430M [INFO] ------------------------------------------------------------------------ Repository: lens Description ------- https://issues.apache.org/jira/browse/LENS-551 - Removed all printStackTrace from project - Replaced by LOG.error(e.getMessage()) - Imposed checkstyle checks for restricting printStackTrace Diffs (updated) ----- checkstyle/src/main/resources/checkstyle.xml 236f977 lens-cli/src/main/java/org/apache/lens/cli/commands/BaseLensCommand.java 8707fec lens-cli/src/main/java/org/apache/lens/cli/commands/LensConnectionCommands.java d727ecc lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionCommands.java 5df3486 lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionTableCommands.java a98f183 lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommands.java 19979cc lens-cli/src/test/java/org/apache/lens/cli/TestLensQueryCommands.java d7b6372 lens-client/src/main/java/org/apache/lens/client/jdbc/LensJdbcDriver.java a6fae87 lens-cube/src/main/java/org/apache/lens/cube/parse/HQLParser.java 1c375ed lens-cube/src/test/java/org/apache/lens/cube/metadata/timeline/TestPartitionTimelines.java 0231e8b lens-cube/src/test/java/org/apache/lens/cube/parse/CubeTestSetup.java 5c776cc lens-cube/src/test/java/org/apache/lens/cube/parse/TestCubeRewriter.java 0f51978 lens-cube/src/test/java/org/apache/lens/cube/parse/TestDateUtil.java a916dbf lens-cube/src/test/java/org/apache/lens/cube/parse/TestHQLParser.java fef634d lens-cube/src/test/java/org/apache/lens/cube/parse/TestMaxUpdateInterval.java a5f64ec lens-cube/src/test/java/org/apache/lens/cube/parse/TestQueryRewrite.java 994a244 lens-cube/src/test/java/org/apache/lens/cube/parse/TestTimeRangeWriter.java 106660e lens-cube/src/test/java/org/apache/lens/cube/parse/TestTimeRangeWriterWithQuery.java b5aaeb2 lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java 43985ba lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveInMemoryResultSet.java a953ec6 lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveQueryPlan.java 9d43fa8 lens-driver-hive/src/test/java/org/apache/lens/driver/hive/TestRemoteHiveDriver.java f0566c8 lens-driver-jdbc/src/test/java/org/apache/lens/driver/jdbc/TestColumnarSQLRewriter.java dc30bb9 lens-driver-jdbc/src/test/java/org/apache/lens/driver/jdbc/TestDataSourceConnectionProvider.java e526700 lens-driver-jdbc/src/test/java/org/apache/lens/driver/jdbc/TestJdbcDriver.java 75c428a lens-examples/src/main/java/org/apache/lens/examples/SampleMetastore.java 2f28e5d lens-ml-lib/src/main/java/org/apache/lens/client/LensMLClient.java 8ffe853 lens-ml-lib/src/main/java/org/apache/lens/ml/algo/lib/AlgoArgParser.java 51979d8 lens-ml-lib/src/main/java/org/apache/lens/ml/impl/LensMLImpl.java e4c03c8 lens-ml-lib/src/test/java/org/apache/lens/ml/TestMLResource.java c08a5b1 lens-server/src/main/java/org/apache/lens/server/LensRequestListener.java 7bf5d09 lens-server/src/main/java/org/apache/lens/server/LensServletContextListener.java 61a0dd8 lens-server/src/main/java/org/apache/lens/server/metastore/CubeMetastoreServiceImpl.java 4597614 lens-server/src/main/java/org/apache/lens/server/metastore/JAXBUtils.java a628c90 lens-server/src/main/java/org/apache/lens/server/metrics/MetricsServiceImpl.java ee24e1f lens-server/src/main/java/org/apache/lens/server/query/LensServerDAO.java a57888d lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 343cd01 lens-server/src/main/java/org/apache/lens/server/query/QueryServiceResource.java 9b6d6bc lens-server/src/main/java/org/apache/lens/server/session/HiveSessionService.java bb0d301 lens-server/src/main/java/org/apache/lens/server/stats/event/query/QueryExecutionStatistics.java 4acf748 lens-server/src/test/java/org/apache/lens/server/LensTestUtil.java aab9771 lens-server/src/test/java/org/apache/lens/server/metastore/TestMetastoreService.java 885a69a lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java dc243aa lens-server/src/test/java/org/apache/lens/server/session/TestDatabaseResourceService.java 640cc41 lens-server/src/test/java/org/apache/lens/server/stats/TestStatisticsLogFileScannerTask.java bf0526f Diff: https://reviews.apache.org/r/34286/diff/ Testing ------- Tested +ve and -ve scenario with project build. [INFO] ------------------------------------------------------------------------ [INFO] Reactor Summary: [INFO] [INFO] Lens Checkstyle Rules .............................. SUCCESS [ 2.242 s] [INFO] Lens ............................................... SUCCESS [ 2.553 s] [INFO] Lens API ........................................... SUCCESS [ 17.961 s] [INFO] Lens API for server and extensions ................. SUCCESS [ 21.129 s] [INFO] Lens Cube .......................................... SUCCESS [02:42 min] [INFO] Lens DB storage .................................... SUCCESS [ 21.738 s] [INFO] Lens Query Library ................................. SUCCESS [ 16.051 s] [INFO] Lens Hive Driver ................................... SUCCESS [02:40 min] [INFO] Lens Driver for JDBC ............................... SUCCESS [ 33.985 s] [INFO] Lens Server ........................................ SUCCESS [05:19 min] [INFO] Lens client ........................................ SUCCESS [ 40.970 s] [INFO] Lens CLI ........................................... SUCCESS [01:57 min] [INFO] Lens Examples ...................................... SUCCESS [ 9.992 s] [INFO] Lens Distribution .................................. SUCCESS [ 7.871 s] [INFO] Lens ML Lib ........................................ SUCCESS [01:18 min] [INFO] Lens ML Ext Distribution ........................... SUCCESS [ 3.419 s] [INFO] Lens Regression .................................... SUCCESS [ 7.997 s] [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 17:04 min [INFO] Finished at: 2015-05-16T00:31:54+05:30 [INFO] Final Memory: 120M/318M [INFO] ------------------------------------------------------------------------ Thanks, Yash Sharma
