[FLUME-987] LoggerSink prints garbage for body

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Duplicate
• Affects Version/s: 1.0.0
• Assignee: Mike Percy
• Reporter: Mike Percy
• Created: 22/Feb/12 01:10
• Updated: 14/Mar/12 05:14
• Resolved: 22/Feb/12 01:39

Description

The LoggerSink relies on Event.toString() being implemented, which is not reliable since it's not part of the Event interface (of course, it's part of Object and prints some memory address by default).

SimpleEvent implements toString() but poorly (prints the memory address for the body byte array), and other implementations of Event may not override toString().

I will submit a patch to remove the toString() method from SimpleEvent and implement a quick fix in LoggerSink.

Longer term, we may want to consider using something like MIME types via the headers so that decoding is more automatic and less reliant on the configured classes being fully cognizant of the data encoding.

Issue Links

• duplicates: FLUME-828 LoggerSink representation of the event's body isn't too useful