You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flume.apache.org by Jeremy Karlson <je...@gmail.com> on 2014/02/27 03:59:28 UTC

Review Request 18555: HTTP source handler doesn't allow for responses

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

Review request for Flume.


Bugs: FLUME-2333
    https://issues.apache.org/jira/browse/FLUME-2333


Repository: flume-git


Description
-------

Add a bidirectional HTTP handler to operate beside the existing one.  This shouldn't break any existing functionality.


Diffs
-----

  flume-ng-core/src/main/java/org/apache/flume/source/http/BidirectionalHTTPSourceHandler.java PRE-CREATION 
  flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSource.java 115b34f 
  flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java 5b07a6e 
  flume-ng-doc/sphinx/FlumeUserGuide.rst 8390cd2 

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


Testing
-------

Unit testing.


Thanks,

Jeremy Karlson

Re: Review Request 18555: HTTP source handler doesn't allow for responses

Posted by Jeremy Karlson <je...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18555/
-----------------------------------------------------------

(Updated March 17, 2014, 7:01 p.m.)


Review request for Flume.


Bugs: FLUME-2333
    https://issues.apache.org/jira/browse/FLUME-2333


Repository: flume-git


Description
-------

Add a bidirectional HTTP handler to operate beside the existing one.  This shouldn't break any existing functionality.


Diffs (updated)
-----

  flume-ng-core/src/main/java/org/apache/flume/source/http/BidirectionalHTTPSourceHandler.java PRE-CREATION 
  flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSource.java 115b34f 
  flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSourceConfigurationConstants.java ed52827 
  flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java 5b07a6e 
  flume-ng-doc/sphinx/FlumeUserGuide.rst 4bcd8a2 

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


Testing
-------

Unit testing.


Thanks,

Jeremy Karlson