[prev in list] [next in list] [prev in thread] [next in thread] 

List:       flume-dev
Subject:    Re: Review Request: Increase JDBC channel test heap size.
From:       "Eric Sammer" <esammer () cloudera ! com>
Date:       2011-11-30 18:14:10
Message-ID: 20111130181410.3286.43294 () reviews ! apache ! org
[Download RAW message or body]


-----------------------------------------------------------
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
> =

>



[prev in list] [next in list] [prev in thread] [next in thread]