----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30390/#review70152 -----------------------------------------------------------
lens-cli/src/test/java/org/apache/lens/cli/TestLensConnectionCliCommands.java <https://reviews.apache.org/r/30390/#comment115160> Can we assert the value also - to include filename? lens-cli/src/test/java/org/apache/lens/cli/TestLensConnectionCliCommands.java <https://reviews.apache.org/r/30390/#comment115163> Can we put this in finally block? lens-server-api/src/main/java/org/apache/lens/server/api/session/SessionService.java <https://reviews.apache.org/r/30390/#comment115166> Can you add javadoc? lens-server/src/main/java/org/apache/lens/server/session/SessionResource.java <https://reviews.apache.org/r/30390/#comment115168> Please add javadoc clearly saying what values type can take. The javadoc here will generate REST api doc as well. lens-server/src/test/java/org/apache/lens/server/session/TestSessionResource.java <https://reviews.apache.org/r/30390/#comment115180> Can you add asserting in testAuxJars as well? - Amareshwari Sriramadasu On Jan. 29, 2015, 2:40 a.m., Raju Bairishetti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30390/ > ----------------------------------------------------------- > > (Updated Jan. 29, 2015, 2:40 a.m.) > > > Review request for lens, Amareshwari Sriramadasu and Jaideep dhok. > > > Bugs: LENS-19 > https://issues.apache.org/jira/browse/LENS-19 > > > Repository: lens > > > Description > ------- > > Provide an option to list all resources added in a session > > > Diffs > ----- > > > lens-cli/src/main/java/org/apache/lens/cli/commands/LensConnectionCommands.java > f1b4ea7 > > lens-cli/src/test/java/org/apache/lens/cli/TestLensConnectionCliCommands.java > c8d0383 > lens-client/src/main/java/org/apache/lens/client/LensClient.java 85329d6 > lens-client/src/main/java/org/apache/lens/client/LensConnection.java > bc07e68 > > lens-server-api/src/main/java/org/apache/lens/server/api/session/SessionService.java > aa65e15 > lens-server/src/main/java/org/apache/lens/server/LensService.java 7756236 > > lens-server/src/main/java/org/apache/lens/server/session/HiveSessionService.java > 7641232 > > lens-server/src/main/java/org/apache/lens/server/session/SessionResource.java > b3cf2c3 > > lens-server/src/test/java/org/apache/lens/server/session/TestSessionResource.java > 109b36d > src/site/apt/user/cli.apt 270265c > > Diff: https://reviews.apache.org/r/30390/diff/ > > > Testing > ------- > > Added a unit test in TestLensConnectionCliCommands and testing th behavior in > TestSessionResource > > > Thanks, > > Raju Bairishetti > >
