----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36178/ -----------------------------------------------------------
(Updated July 15, 2015, 5:33 p.m.) Review request for lens. Changes ------- Implemented review comments Repository: lens Description ------- Added changes for file name suggestions on Tab. Changed String file paths to File. https://issues.apache.org/jira/browse/LENS-634 Diffs (updated) ----- lens-cli/src/main/java/org/apache/lens/cli/commands/BaseLensCommand.java 74b4e91 lens-cli/src/main/java/org/apache/lens/cli/commands/LensCRUDCommand.java 9cb7d69 lens-cli/src/main/java/org/apache/lens/cli/commands/LensConnectionCommands.java 8b4face lens-cli/src/main/java/org/apache/lens/cli/commands/LensCubeCommands.java 6ba702f lens-cli/src/main/java/org/apache/lens/cli/commands/LensDimensionCommands.java de022c1 lens-cli/src/main/java/org/apache/lens/cli/commands/LensDimensionTableCommands.java 6a93393 lens-cli/src/main/java/org/apache/lens/cli/commands/LensFactCommands.java bdb9c38 lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java c48fabd lens-cli/src/main/java/org/apache/lens/cli/commands/LensStorageCommands.java 928120a lens-cli/src/main/java/org/apache/lens/cli/commands/PhysicalTableCrudCommand.java a479c14 lens-cli/src/test/java/org/apache/lens/cli/TestLensConnectionCliCommands.java 7503221 lens-cli/src/test/java/org/apache/lens/cli/TestLensCubeCommands.java 73661e1 lens-cli/src/test/java/org/apache/lens/cli/TestLensDatabaseCommands.java b6f96e6 lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionCommands.java a22862e lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionTableCommands.java a87d0b7 lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommands.java 13bfbd2 lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommandsWithMissingWeight.java 73f3a78 lens-cli/src/test/java/org/apache/lens/cli/TestLensQueryCommands.java 46e28a9 lens-cli/src/test/java/org/apache/lens/cli/TestLensStorageCommands.java 3a04ec6 lens-server/src/main/java/org/apache/lens/server/util/ScannedPaths.java 92687c3 src/site/apt/user/cli.apt 353c171 Diff: https://reviews.apache.org/r/36178/diff/ Testing ------- New build results- [INFO] ------------------------------------------------------------------------ [INFO] Reactor Summary: [INFO] [INFO] Lens Checkstyle Rules .............................. SUCCESS [ 2.460 s] [INFO] Lens ............................................... SUCCESS [ 2.842 s] [INFO] Lens API ........................................... SUCCESS [ 21.035 s] [INFO] Lens API for server and extensions ................. SUCCESS [ 21.545 s] [INFO] Lens Cube .......................................... SUCCESS [03:04 min] [INFO] Lens DB storage .................................... SUCCESS [ 19.023 s] [INFO] Lens Query Library ................................. SUCCESS [ 14.263 s] [INFO] Lens Hive Driver ................................... SUCCESS [02:28 min] [INFO] Lens Driver for JDBC ............................... SUCCESS [ 34.581 s] [INFO] Lens Server ........................................ SUCCESS [05:10 min] [INFO] Lens client ........................................ SUCCESS [ 36.137 s] [INFO] Lens CLI ........................................... SUCCESS [02:06 min] [INFO] Lens Examples ...................................... SUCCESS [ 9.193 s] [INFO] Lens Distribution .................................. SUCCESS [ 7.860 s] [INFO] Lens ML Lib ........................................ SUCCESS [01:19 min] [INFO] Lens ML Ext Distribution ........................... SUCCESS [ 3.114 s] [INFO] Lens Regression .................................... SUCCESS [ 12.437 s] [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 17:15 min [INFO] Finished at: 2015-07-05T00:12:45+05:30 [INFO] Final Memory: 125M/375M [INFO] ------------------------------------------------------------------------ Thanks, Yash Sharma
