----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16516/#review85383 -----------------------------------------------------------
I took a look, this looks good to me. Since Johny has reviewed it more thoroughly, I am going to run tests and commit this one for now. - Hari Shreedharan On Dec. 30, 2013, 4:40 p.m., Alexandre Dutra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16516/ > ----------------------------------------------------------- > > (Updated Dec. 30, 2013, 4:40 p.m.) > > > Review request for Flume. > > > Repository: flume-git > > > Description > ------- > > See https://issues.apache.org/jira/browse/FLUME-2215 for details. > > Please note that I am actually proposing two different patches to fix this > issue. Reviewers please test them separately. > > > Diffs > ----- > > > flume-ng-core/src/main/java/org/apache/flume/serialization/ResettableFileInputStream.java > 587ab29 > > flume-ng-core/src/test/java/org/apache/flume/serialization/TestResettableFileInputStream.java > 73df9c9 > > Diff: https://reviews.apache.org/r/16516/diff/ > > > Testing > ------- > > > Thanks, > > Alexandre Dutra > >
