> On Dec. 22, 2015, 8:59 a.m., Amareshwari Sriramadasu wrote: > > lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java, > > line 289 > > <https://reviews.apache.org/r/41033/diff/8/?file=1173786#file1173786line289> > > > > seems doc need change. By default async is false - unspecified default > > value is true. > > Rajat Khandelwal wrote: > if unspecified value is true then the default is true. unspecified value > is the value assumed when value is not provided for this argument.
Rajat, What i meant here is the following : If we do : * "query execute <query-string> " # excutes synchronously. As the default value for async is false * "query execute <query-string> --async" # executes asynchronously. As the unspecified default value for async is true * "query execute <query-string> --async true" # executes asynchronously. value is specified explicitly. * "query execute <query-string> --async false" # executes synchronously. value is specified explicitly. Please validate the above and update doc accordingly. - Amareshwari ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41033/#review111586 ----------------------------------------------------------- On Dec. 23, 2015, 5:04 a.m., Rajat Khandelwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41033/ > ----------------------------------------------------------- > > (Updated Dec. 23, 2015, 5:04 a.m.) > > > Review request for lens. > > > Bugs: LENS-836 > https://issues.apache.org/jira/browse/LENS-836 > > > Repository: lens > > > Description > ------- > > > Diffs > ----- > > lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java > e3c08ff1e17feeb6b7d3a5e42599993350e7d166 > lens-cli/src/test/java/org/apache/lens/cli/TestLensQueryCommands.java > 6ebfff7f47a8c98350dbc9466f86312334898b15 > lens-client/src/main/java/org/apache/lens/client/LensClient.java > 8f197e4ec5f9c0cc050643cf7333aeba5eec6e08 > lens-client/src/main/java/org/apache/lens/client/LensStatement.java > 71caa48e7926e6d0f9b486ece71b292ba598be3c > src/site/apt/user/cli.apt 3db53c44900176dc7f062267a200ec5c8ced5f6a > > Diff: https://reviews.apache.org/r/41033/diff/ > > > Testing > ------- > > http://showterm.io/2bbd961fd3bcc859adbc8 > > > Thanks, > > Rajat Khandelwal > >
