-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3723/
-----------------------------------------------------------

(Updated 2012-04-20 01:05:11.472519)


Review request for Flume.


Changes
-------

Here is an updated patch adressing Arvind's concerns.
Let me know if more work need to be done


Summary
-------

Here is a review form a rebased patch of FLUME-923.
Sorry Dave, reviewboard didn't want your patch. So I re-exported from my branch


This addresses bug FLUME-923.
    https://issues.apache.org/jira/browse/FLUME-923


Diffs (updated)
-----

  flume-ng-sinks/pom.xml acb3087 
  pom.xml ed8092d 
  
flume-ng-sinks/flume-jms-sink/src/test/java/org/apache/flume/sink/jms/TestJMSSink.java
 PRE-CREATION 
  flume-ng-node/pom.xml da0d15e 
  flume-ng-sinks/flume-jms-sink/pom.xml PRE-CREATION 
  
flume-ng-sinks/flume-jms-sink/src/main/java/org/apache/flume/sink/jms/JMSSink.java
 PRE-CREATION 
  flume-ng-core/src/main/java/org/apache/flume/sink/SinkType.java 6b08c09 
  flume-ng-dist/pom.xml 642e681 

Diff: https://reviews.apache.org/r/3723/diff


Testing
-------


Thanks,

Bruno

Reply via email to