[GitHub] [flume] rgoers opened a new pull request #347: FLUME-3335 - Support configuration via HTTP(S)

[GitBox <gi...@apache.org>]

rgoers opened a new pull request #347:
URL: https://github.com/apache/flume/pull/347


   This PR includes the following changes: 
   
   1. Adds support for configuration via a URI. The currently supported protocols are HTTTP/S, file, and classpath. File and HTTP support reloading. It supports the use of Basic Authentication out of the box but a custom Authentication class can be provided.
   2. Supports multiple configuration files. The files are merged into one configuration. This allows a base configuration and different files for prod or dev, etc.
   3. Uses Apache Commons Text's StringSubstitutor to allow variable replacement. So values can include ${env:PORT}, ${sys:port}, or just ${PORT} depending on the need. ${PORT} would reference a property in one of the properties files (presumably an override file). 
   
   When building all the existing tests passed except for the TestSpooldirSource in the integration tests. However, that test failed for me before I made any changes. I will look into that failure separately.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@flume.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [flume] rgoers merged pull request #347: FLUME-3335 - Support configuration via HTTP(S)

[GitBox <gi...@apache.org>]

rgoers merged pull request #347:
URL: https://github.com/apache/flume/pull/347


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@flume.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org