----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15107/#review28135 -----------------------------------------------------------
flume-ng-sinks/flume-ng-morphline-solr-sink/src/main/java/org/apache/flume/sink/solr/morphline/BlobHandler.java <https://reviews.apache.org/r/15107/#comment54748> How about using the same implementation approach as for BlobDeserializer wrt. subclassing, etc? - Wolfgang Hoschek On Oct. 31, 2013, 12:48 a.m., Roshan Naik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15107/ > ----------------------------------------------------------- > > (Updated Oct. 31, 2013, 12:48 a.m.) > > > Review request for Flume. > > > Bugs: FLUME-2226 > https://issues.apache.org/jira/browse/FLUME-2226 > > > Repository: flume-git > > > Description > ------- > > - Moved BlobHandler out of morphline sink and into HTTP source along with > tests. > - Updated docs to reflect new FQCN > - Retained dummy class for old FQCN compat > > > Diffs > ----- > > flume-ng-core/src/main/java/org/apache/flume/source/http/BlobHandler.java > PRE-CREATION > > flume-ng-core/src/test/java/org/apache/flume/source/http/FlumeHttpServletByteRequestWrapper.java > PRE-CREATION > > flume-ng-core/src/test/java/org/apache/flume/source/http/TestBlobHandler.java > PRE-CREATION > flume-ng-doc/sphinx/FlumeUserGuide.rst e38bb67 > > flume-ng-sinks/flume-ng-morphline-solr-sink/src/main/java/org/apache/flume/sink/solr/morphline/BlobHandler.java > e84dec1 > > flume-ng-sinks/flume-ng-morphline-solr-sink/src/test/java/org/apache/flume/sink/solr/morphline/FlumeHttpServletRequestWrapper.java > 9711a3a > > flume-ng-sinks/flume-ng-morphline-solr-sink/src/test/java/org/apache/flume/sink/solr/morphline/TestBlobHandler.java > 3e7de99 > > Diff: https://reviews.apache.org/r/15107/diff/ > > > Testing > ------- > > Ran unit tests & some manual Test. > > > Thanks, > > Roshan Naik > >