----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51756/#review148713 -----------------------------------------------------------
lens-client/src/main/java/org/apache/lens/client/LensClient.java (line 120) <https://reviews.apache.org/r/51756/#comment216215> Just had one thought. Is there a way to keep the number of public methods to a minimum in LensClient and LensStatement. The motivation for this would be have lesser number of execute methods to choose from for end user who sometimes seem to struggle which method to use. I Dont see a direct way of doing that unless we sacrifice backward compatibility. - Puneet Gupta On Sept. 9, 2016, 11:09 a.m., Srikanth Sundarrajan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51756/ > ----------------------------------------------------------- > > (Updated Sept. 9, 2016, 11:09 a.m.) > > > Review request for lens. > > > Bugs: LENS-1319 > https://issues.apache.org/jira/browse/LENS-1319 > > > Repository: lens > > > Description > ------- > > LENS-1319 Lens Client doesn't provide the option to pass query conf while > submitting the query > > > Diffs > ----- > > lens-client/src/main/java/org/apache/lens/client/LensClient.java 9a76118 > lens-client/src/main/java/org/apache/lens/client/LensStatement.java 64c24c7 > lens-client/src/test/java/org/apache/lens/client/TestLensClient.java > 94a788a > > Diff: https://reviews.apache.org/r/51756/diff/ > > > Testing > ------- > > Relevant unit tests added. > > > Thanks, > > Srikanth Sundarrajan > >
