-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50378/
-----------------------------------------------------------

(Updated 三月 13, 2017, 9 a.m.)


Review request for Flume.


Changes
-------

Unify regex. Add two config FILE_GROUPS_SUFFIX_DIR and FILE_GROUPS_SUFFIX_FILE.


Repository: flume-git


Description
-------

In our log management project, we wan't to track many log files like this:
/app/dir1/log.*
/app/dir2/log.*
...
/app/dirn/log.*
But TaildirSource can't support wildcards in filegroup directory name. The 
following config is expected:
a1.sources.r1.filegroups.fg = /app/*/log.*


Diffs (updated)
-----

  flume-ng-doc/sphinx/FlumeUserGuide.rst d863068 
  
flume-ng-sources/flume-taildir-source/src/main/java/org/apache/flume/source/taildir/ReliableTaildirEventReader.java
 8838320 
  
flume-ng-sources/flume-taildir-source/src/main/java/org/apache/flume/source/taildir/TaildirMatcher.java
 ad9f720 
  
flume-ng-sources/flume-taildir-source/src/main/java/org/apache/flume/source/taildir/TaildirSource.java
 a107a01 
  
flume-ng-sources/flume-taildir-source/src/main/java/org/apache/flume/source/taildir/TaildirSourceConfigurationConstants.java
 f2347f3 
  
flume-ng-sources/flume-taildir-source/src/test/java/org/apache/flume/source/taildir/TestTaildirEventReader.java
 bcfe4bb 
  
flume-ng-sources/flume-taildir-source/src/test/java/org/apache/flume/source/taildir/TestTaildirMatcher.java
 c341054 
  
flume-ng-sources/flume-taildir-source/src/test/java/org/apache/flume/source/taildir/TestTaildirSource.java
 097ee0b 


Diff: https://reviews.apache.org/r/50378/diff/6/

Changes: https://reviews.apache.org/r/50378/diff/5-6/


Testing
-------

All tests in TestTaildirSource passed.


Thanks,

qiao wen

Reply via email to