----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33976/#review83541 -----------------------------------------------------------
lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java <https://reviews.apache.org/r/33976/#comment134544> Can we make sure handle is always closed? Can we move close() method call to finally block? - Raju Bairishetti On May 12, 2015, 1:59 p.m., Rajat Khandelwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33976/ > ----------------------------------------------------------- > > (Updated May 12, 2015, 1:59 p.m.) > > > Review request for lens. > > > Bugs: LENS-26 > https://issues.apache.org/jira/browse/LENS-26 > > > Repository: lens > > > Description > ------- > > option to download result into a file > > > Diffs > ----- > > .reviewboardrc e9a57e0 > lens-cli/src/main/java/org/apache/lens/cli/commands/LensQueryCommands.java > c0616d2 > lens-cli/src/test/java/org/apache/lens/cli/TestLensQueryCommands.java > d7b6372 > src/site/apt/user/cli.apt cbbc3eb > > Diff: https://reviews.apache.org/r/33976/diff/ > > > Testing > ------- > > None > > > Thanks, > > Rajat Khandelwal > >
