----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10607/#review19439 -----------------------------------------------------------
Ship it! Good catch. - Alexander Alten-Lorenz On April 18, 2013, 4:35 p.m., Paul Chavez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10607/ > ----------------------------------------------------------- > > (Updated April 18, 2013, 4:35 p.m.) > > > Review request for Flume. > > > Description > ------- > > After checking that the meta file is tracking the right target, don't create > a new instance unless it's needed. Otherwise a new instance is requested > while the old one is still open for read/write which triggers access > exception in Windows. > > > This addresses bug FLUME-1993. > https://issues.apache.org/jira/browse/FLUME-1993 > > > Diffs > ----- > > > flume-ng-core/src/main/java/org/apache/flume/client/avro/ReliableSpoolingFileEventReader.java > 37f0ee1 > > Diff: https://reviews.apache.org/r/10607/diff/ > > > Testing > ------- > > Unit tests not run. Functional testing for this specific issue has been done > on Windows Server 2008 R2 > > > Thanks, > > Paul Chavez > >
