> On April 22, 2016, 8:09 a.m., Rajat Khandelwal wrote: > > lens-client/src/main/java/org/apache/lens/client/resultset/ResultSet.java, > > line 29 > > <https://reviews.apache.org/r/46485/diff/1/?file=1354881#file1354881line29> > > > > I think this can also implement `Iterator<String[]>` which will imply > > that the syntax like the following is supported: > > > > ``` > > ResultSet rs = ...; > > for(String[] line: rs) { > > ... > > } > > ``` > > Puneet Gupta wrote: > The Iterator syntax which you suggested looks tempting. > > Looked at java.util.Iterator . It has more methods which will not make > sense (remove, forEachRemaining). More methods can be added in future too. > Want to stick to booelan next() and String[] getRow() and not expose other > dummy methods from iterator.
Execption handling in Iterator interface is also a blocker. Will have to throw a RunTimeExecption - Puneet ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46485/#review130059 ----------------------------------------------------------- On April 25, 2016, 11:12 a.m., Puneet Gupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46485/ > ----------------------------------------------------------- > > (Updated April 25, 2016, 11:12 a.m.) > > > Review request for lens. > > > Bugs: LENS-1008 > https://issues.apache.org/jira/browse/LENS-1008 > > > Repository: lens > > > Description > ------- > > The result iterator can have the following methods > public interface ResultSet > { public boolean hasNext(); public String[] next(); public String[] > getColumnNames(); } > This can be used by clients to programmatically consume the result with ease. > > > Have supported Zipped CSV abd CSV for now with configurable delimiter, > encoding and isHeaderPresent. > It can be extend to other formats in future > > Users can access this feature via easily via (Another varaiation avialble > with more options) > LensClient.getCsvResultSet(QueryHandle) > LensClient.getZippedCsvResultSet(QueryHandle) > > > Diffs > ----- > > lens-client/pom.xml 4fd01fb > lens-client/src/main/java/org/apache/lens/client/LensClient.java 9626820 > lens-client/src/main/java/org/apache/lens/client/LensConnection.java > d67e64e > > lens-client/src/main/java/org/apache/lens/client/exceptions/LensClientIOException.java > PRE-CREATION > > lens-client/src/main/java/org/apache/lens/client/resultset/AbstractResultSet.java > PRE-CREATION > > lens-client/src/main/java/org/apache/lens/client/resultset/CsvResultSet.java > PRE-CREATION > > lens-client/src/main/java/org/apache/lens/client/resultset/CsvResultSetReader.java > PRE-CREATION > lens-client/src/main/java/org/apache/lens/client/resultset/ResultSet.java > PRE-CREATION > > lens-client/src/main/java/org/apache/lens/client/resultset/ResultSetReader.java > PRE-CREATION > > lens-client/src/main/java/org/apache/lens/client/resultset/ZippedCsvResultSet.java > PRE-CREATION > > lens-client/src/main/java/org/apache/lens/client/resultset/ZippedCsvResultSetReader.java > PRE-CREATION > > lens-client/src/main/java/org/apache/lens/client/resultset/ZippedResultSetReader.java > PRE-CREATION > lens-client/src/test/java/org/apache/lens/client/TestLensClient.java > 7a00f65 > lens-client/src/test/resources/dim2-part/data.data PRE-CREATION > lens-client/src/test/resources/dim_table.xml PRE-CREATION > lens-client/src/test/resources/fact1.xml PRE-CREATION > lens-client/src/test/resources/lens-client-site.xml b356e5e > lens-client/src/test/resources/local-storage.xml PRE-CREATION > lens-client/src/test/resources/sample-cube.xml PRE-CREATION > lens-client/src/test/resources/test-detail.xml PRE-CREATION > lens-client/src/test/resources/test-dimension.xml PRE-CREATION > > Diff: https://reviews.apache.org/r/46485/diff/ > > > Testing > ------- > > ------------------------------------------------------- > T E S T S > ------------------------------------------------------- > Listening for transport dt_socket at address: 8000 > Running org.apache.lens.client.TestLensClient > Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 145.247 sec - > in org.apache.lens.client.TestLensClient > > Results : > > Tests run: 7, Failures: 0, Errors: 0, Skipped: 0 > > > Thanks, > > Puneet Gupta > >
