----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3928/#review6201 -----------------------------------------------------------
Ship it! +1 Thanks for the patch Brock. Can you please rebase it to the trunk and attache it to the Jira. - Arvind On 2012-02-28 13:46:28, Brock Noland wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3928/ > ----------------------------------------------------------- > > (Updated 2012-02-28 13:46:28) > > > Review request for Flume. > > > Summary > ------- > > Changes SimpleEvent.toString() to include a human readable representation of > the body byte array. > > > This addresses bug FLUME-828. > https://issues.apache.org/jira/browse/FLUME-828 > > > Diffs > ----- > > flume-ng-core/src/test/java/org/apache/flume/event/TestSimpleEvent.java > PRE-CREATION > flume-ng-core/src/main/java/org/apache/flume/event/SimpleEvent.java e0c3b45 > > Diff: https://reviews.apache.org/r/3928/diff > > > Testing > ------- > > Added two tests for the new code. > > > Thanks, > > Brock > >
