-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3505/
-----------------------------------------------------------
(Updated 2012-01-18 23:08:42.213397)
Review request for Flume.
Changes
-------
Here is an updated patch to be applied on top of the previous one
Summary
-------
Add an implementation for a JMS sink
This addresses bug FLUME-923.
https://issues.apache.org/jira/browse/FLUME-923
Diffs (updated)
-----
flume-ng-sinks/flume-jms-sink/src/test/java/org/apache/flume/sink/jms/TestJMSSink.java
PRE-CREATION
flume-ng-core/src/main/java/org/apache/flume/sink/SinkType.java 5be602a
flume-ng-sinks/flume-jms-sink/src/main/java/org/apache/flume/sink/jms/JMSSink.java
PRE-CREATION
Diff: https://reviews.apache.org/r/3505/diff
Testing
-------
* Checked unit tests pass, including the one from the JMS sink
* Brought up my own external JMS broker and hooked up a JMS sink on a flume node
Thanks,
Bruno
