----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2930/#review3582 -----------------------------------------------------------
Ship it! lgtm! - Eric On 2011-11-24 20:22:06, Arvind Prabhakar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2930/ > ----------------------------------------------------------- > > (Updated 2011-11-24 20:22:06) > > > Review request for Flume and Eric Sammer. > > > Summary > ------- > > Fix for FLUME-816 > > > This addresses bug FLUME-816. > https://issues.apache.org/jira/browse/FLUME-816 > > > Diffs > ----- > > .gitignore c319325 > flume-ng-channels/flume-jdbc-channel/pom.xml 8665eee > > Diff: https://reviews.apache.org/r/2930/diff > > > Testing > ------- > > Ran all JDBC channel tests on a different system where I was able to see the > OOM errors. With this change, the tests pass. > > > Thanks, > > Arvind > >
