> On Jan. 9, 2015, 9:29 a.m., Amareshwari Sriramadasu wrote: > > lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/JDBCDriver.java, > > line 358 > > <https://reviews.apache.org/r/29758/diff/3/?file=814206#file814206line358> > > > > Fetch size set at result set retrieval will override this? If so, can > > you update the config doc with the same?
Both Statement and ResultSet have setFetchSize calls. I couldn't confirm from JDBC doc if setting fetch size an the result set will override statement result set. - Jaideep ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29758/#review67399 ----------------------------------------------------------- On Jan. 12, 2015, 7:38 a.m., Jaideep dhok wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29758/ > ----------------------------------------------------------- > > (Updated Jan. 12, 2015, 7:38 a.m.) > > > Review request for lens and Amareshwari Sriramadasu. > > > Bugs: lens-188 > https://issues.apache.org/jira/browse/lens-188 > > > Repository: lens > > > Description > ------- > > Added two new configs, one for fetch size and one to enable streaming result > sets. > > > Diffs > ----- > > lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/JDBCDriver.java > 09f88d8 > > lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/JDBCDriverConfConstants.java > 489af81 > lens-driver-jdbc/src/main/resources/jdbcdriver-default.xml 4f0acc7 > src/site/apt/admin/jdbcdriver-config.apt 54e5dd0 > > Diff: https://reviews.apache.org/r/29758/diff/ > > > Testing > ------- > > Added a unit test to validate streaming result set config works correctly. > > Validated existing tests in JDBC module. > > > Thanks, > > Jaideep dhok > >
