> On 2012-03-30 08:33:50, Mike Percy wrote: > > Since we are catching Throwable here, we may want to do it for FLUME-1074 > > too. Shouldn't we do this in all places where Transaction.commit() is > > called, if for nothing else than consistency?
+1 - Brock ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4569/#review6570 ----------------------------------------------------------- On 2012-03-30 05:38:44, Prasad Mujumdar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4569/ > ----------------------------------------------------------- > > (Updated 2012-03-30 05:38:44) > > > Review request for Flume, Arvind Prabhakar and Hari Shreedharan. > > > Summary > ------- > > Unhandled thorwable in HDFS sink can leave transaction open. > > > This addresses bug FLUME-1075. > https://issues.apache.org/jira/browse/FLUME-1075 > > > Diffs > ----- > > > flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/HDFSEventSink.java > eee9221 > > Diff: https://reviews.apache.org/r/4569/diff > > > Testing > ------- > > regression tests > > > Thanks, > > Prasad > >
