You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flume.apache.org by Johny Rufus John <jr...@cloudera.com> on 2015/05/26 19:55:45 UTC

Re: Review Request 16516: [FLUME-2215] ResettableFileInputStream can't support ucs-4 character

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