----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16516/#review85206 -----------------------------------------------------------
+1, Reviewed and tried the patch, solves the 4-byte utf-8 decoding issue - Johny Rufus John 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 > >
