Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Flume, mail # dev - Review Request 12803: Fix for Flume-2109, Take 3. Incorporated Review comments and implementation in sysc with Avro source


Copy link to this message
-
Re: Review Request 12803: Fix for Flume-2109, Take 3. Incorporated Review comments and implementation in sysc with Avro source
Hari Shreedharan 2013-08-02, 05:12
Ashish,

Please attach the patch to the jira. I will commit it.

Hari
Thanks,
Hari
On Thursday, August 1, 2013 at 7:05 PM, Hari Shreedharan wrote:

> This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12803/
>
>
>
>
> Ship it!
> As discussed, please file a follow up jira to add a new test. please update the log message to INFO and attach the patch to the jira.
>
> - Hari Shreedharan
> On July 22nd, 2013, 7:56 a.m. UTC, Ashish Paliwal wrote:
> Review request for Flume.
> By Ashish Paliwal.
>
> Updated July 22, 2013, 7:56 a.m.
>
> Bugs: FLUME-2109 (https://issues.apache.org/jira/browse/FLUME-2109)
> Repository: flume-git
> Description
> Fix for Flume-2109, to support HTTPS. Incorporates Review comments from Alex Implementation refined to be as close to Avro Source, like SSL port is not to be defined, the port value shall be used Documentation updated to reflect the changes
>
>
>
>
> Testing
> Unit Test case written to test SSL functionality
>
>
>
>
> Diffs
> flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSource.java (c90f067)
> flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSourceConfigurationConstants.java (f547e0f)
> flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java (8952db3)
> flume-ng-doc/sphinx/FlumeUserGuide.rst (63cad21)
>
> View Diff (https://reviews.apache.org/r/12803/diff/)
>
>
>
>
>
>
>