----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34797/#review85952 -----------------------------------------------------------
lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java <https://reviews.apache.org/r/34797/#comment137777> Let's keep the `help` of `CliOption` consise, as it goes in `cli.apt` as the format of command. Just `<async>` should do. And provide the description of the new arguemnt in help of `CliCommand`. This is a pattern followed for other commands too. The main aim of `help` options is to have a good user documentation. You can run `mvn site:run` and look at `localhost:8080/user/cli.html` to see how the help for the whole command looks. - Rajat Khandelwal On June 1, 2015, 1 p.m., Pranav Agarwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34797/ > ----------------------------------------------------------- > > (Updated June 1, 2015, 1 p.m.) > > > Review request for lens. > > > Bugs: LENS-590 > https://issues.apache.org/jira/browse/LENS-590 > > > Repository: lens > > > Description > ------- > > lens-590 > > > per review comments from rajat: added cli.apt and adjusted cli params for > query results commands > > > Diffs > ----- > > lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java > 6c4a1b5fc82a8fe4776919546d6f8c975ae0f514 > lens-cli/src/test/java/org/apache/lens/cli/TestLensQueryCommands.java > 32a89ac98230d642738e1954b965c692c7235507 > lens-client/src/main/java/org/apache/lens/client/LensClient.java > cd8bf166e11786ae8ec26df8d03dd381ef62d787 > lens-client/src/main/java/org/apache/lens/client/LensStatement.java > 4472a80b9e8b8b7243ec6d15a854867bb24dcce1 > src/site/apt/user/cli.apt 2016bfddf38751fd98bedb8163d61c6af5b99be3 > > Diff: https://reviews.apache.org/r/34797/diff/ > > > Testing > ------- > > tests have come out clean > > > Thanks, > > Pranav Agarwal > >
