[
https://issues.apache.org/jira/browse/FLUME-1181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13269052#comment-13269052
]
[email protected] commented on FLUME-1181:
------------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5017/
-----------------------------------------------------------
(Updated 2012-05-05 20:18:36.840078)
Review request for Flume, Brock Noland and Hari Shreedharan.
Changes
-------
Thanks Hari and Brock for the review and feedback. This diff incorporates the
feedback and rebases the patch on the latest trunk.
Summary
-------
This change:
* introduces a precondition check in Context.getSubProperties(String prefix)
method enforce that the prefix ends with a period,
* fixes the TestPropertiesFileConfigurationProvider to load the property file
which was previously not working, and fixes cases that are broken
* refactors some of the source (not all) to externalize the configuration keys
into separate constants class
This addresses bug FLUME-1181.
https://issues.apache.org/jira/browse/FLUME-1181
Diffs (updated)
-----
/trunk/flume-ng-node/src/test/resources/flume-conf.properties 1334473
/trunk/flume-ng-node/src/main/java/org/apache/flume/conf/properties/PropertiesFileConfigurationProvider.java
1334473
/trunk/flume-ng-node/src/test/java/org/apache/flume/conf/properties/TestPropertiesFileConfigurationProvider.java
1334473
/trunk/flume-ng-core/src/main/java/org/apache/flume/source/SyslogUDPSource.java
1334473
/trunk/flume-ng-core/src/main/java/org/apache/flume/source/SyslogUtils.java
1334473
/trunk/flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java
1334473
/trunk/flume-ng-core/src/main/java/org/apache/flume/channel/ChannelSelectorFactory.java
1334473
/trunk/flume-ng-core/src/main/java/org/apache/flume/channel/MultiplexingChannelSelector.java
1334473
/trunk/flume-ng-core/src/main/java/org/apache/flume/source/SyslogSourceConfigurationConstants.java
PRE-CREATION
/trunk/flume-ng-configuration/src/main/java/org/apache/flume/conf/source/SourceConfiguration.java
1334473
/trunk/flume-ng-configuration/src/main/java/org/apache/flume/conf/sink/SinkGroupConfiguration.java
1334473
/trunk/flume-ng-configuration/src/main/java/org/apache/flume/Context.java
1334473
/trunk/flume-ng-configuration/src/main/java/org/apache/flume/conf/BasicConfigurationConstants.java
PRE-CREATION
/trunk/flume-ng-configuration/src/main/java/org/apache/flume/conf/ComponentConfiguration.java
1334473
/trunk/flume-ng-configuration/src/main/java/org/apache/flume/conf/FlumeConfiguration.java
1334473
Diff: https://reviews.apache.org/r/5017/diff
Testing
-------
Ran all tests. The updated TestPropertiesFileConfigurationProvider catches two
issues now - the one with channel selector configuration not being set
correctly, and the other a similar issue with syslog source format
configuration. Both of these issues have been fixed with the changes.
Also done some manual validation of the system with a few simple scenarios.
Thanks,
Arvind
> Context must enforce dot-separated prefix for sub-properties.
> -------------------------------------------------------------
>
> Key: FLUME-1181
> URL: https://issues.apache.org/jira/browse/FLUME-1181
> Project: Flume
> Issue Type: Bug
> Components: Configuration
> Reporter: Arvind Prabhakar
> Assignee: Arvind Prabhakar
> Fix For: v1.2.0
>
> Attachments: FLUME-1181-1.patch
>
>
> Currently the method Context.getSubProperties(String prefix) assumes a
> well-formed prefix. This can lead to undetected issues such as what is being
> seen in the current state of channel selector configuration, where the keys
> within the channel selector are being computed as prefixed with a period.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
