Review Board 1.7.22


Fix for Flume-2109, Take 3. Incorporated Review comments and implementation in sysc with Avro source

Review Request #12803 - Created July 22, 2013 and updated

Ashish Paliwal
FLUME-2109
Reviewers
Flume
flume-git
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
Unit Test case written to test SSL functionality
Total:
2
Open:
2
Resolved:
0
Dropped:
0
Status:
From:
Description From Last Updated Status
We should probably log this at INFO level. Hari Shreedharan Aug. 1, 2013, 10:31 p.m. Open
Can you add one test where a non-SSL connection fails? Hari Shreedharan Aug. 1, 2013, 10:31 p.m. Open
Posted (Aug. 1, 2013, 10:31 p.m.)
Ashish,

This looks ready to commit. I have a couple of requests - if you want me to commit this one, and complete the test in a follow-up jira, that is fine by me too. 
We should probably log this at INFO level.
Can you add one test where a non-SSL connection fails?
Ship it!
Posted (Aug. 2, 2013, 2:05 a.m.)
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.