----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29758/#review67634 -----------------------------------------------------------
lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/JDBCDriver.java <https://reviews.apache.org/r/29758/#comment111707> why do we need to set it to Integer.MIN(2^-31) here. Isnt the default of 0 fine here ? - Suma Shivaprasad 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 > >
