> On 2012-05-05 15:14:29, Brock Noland wrote: > > Other than the items Hair mentioned, it looks good to me!
s/Hari/Hari ;) - Hari ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5017/#review7603 ----------------------------------------------------------- On 2012-05-05 06:22:46, Arvind Prabhakar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5017/ > ----------------------------------------------------------- > > (Updated 2012-05-05 06:22:46) > > > Review request for Flume, Brock Noland and Hari Shreedharan. > > > 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 > ----- > > /trunk/flume-ng-configuration/src/main/java/org/apache/flume/Context.java > 1334310 > > /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 > 1334310 > > /trunk/flume-ng-configuration/src/main/java/org/apache/flume/conf/FlumeConfiguration.java > 1334310 > > /trunk/flume-ng-configuration/src/main/java/org/apache/flume/conf/sink/SinkGroupConfiguration.java > 1334310 > > /trunk/flume-ng-configuration/src/main/java/org/apache/flume/conf/source/SourceConfiguration.java > 1334310 > > /trunk/flume-ng-core/src/main/java/org/apache/flume/channel/ChannelSelectorFactory.java > 1334310 > > /trunk/flume-ng-core/src/main/java/org/apache/flume/channel/MultiplexingChannelSelector.java > 1334310 > > /trunk/flume-ng-core/src/main/java/org/apache/flume/source/SyslogSourceConfigurationConstants.java > PRE-CREATION > > /trunk/flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java > 1334310 > > /trunk/flume-ng-core/src/main/java/org/apache/flume/source/SyslogUDPSource.java > 1334310 > /trunk/flume-ng-core/src/main/java/org/apache/flume/source/SyslogUtils.java > 1334310 > > /trunk/flume-ng-node/src/main/java/org/apache/flume/conf/properties/PropertiesFileConfigurationProvider.java > 1334310 > > /trunk/flume-ng-node/src/test/java/org/apache/flume/conf/properties/TestPropertiesFileConfigurationProvider.java > 1334310 > /trunk/flume-ng-node/src/test/resources/flume-conf.properties 1334310 > > 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 > >
