----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12211/ -----------------------------------------------------------
(Updated July 25, 2013, 6:40 p.m.) Review request for Flume. Changes ------- there were some whitespace changes that I missed, uploaded a new DIFF Bugs: https://issues.apache.org/jira/browse/FLUME-1899 https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/FLUME-1899 Repository: flume-git Description ------- Added recursive directory walking to the ReliableSpoolingFileEventReader. the same ignore pattern is used against the sub-directory names as well. Diffs (updated) ----- flume-ng-core/src/main/java/org/apache/flume/client/avro/ReliableSpoolingFileEventReader.java f82fe1f flume-ng-core/src/main/java/org/apache/flume/source/SpoolDirectorySource.java 7145580 flume-ng-core/src/main/java/org/apache/flume/source/SpoolDirectorySourceConfigurationConstants.java f3cc703 flume-ng-core/src/test/java/org/apache/flume/source/TestSpoolDirectorySource.java 652d2a2 flume-ng-doc/sphinx/FlumeUserGuide.rst f84cda5 Diff: https://reviews.apache.org/r/12211/diff/ Testing ------- Added new unit test, local testing on windows and CentOS. By default the setting is not enabled, need to update configuration i.e. spool-agent.sources.spooler.recursiveDirectorySearch = true Thanks, Phil Scala
