You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flume.apache.org by Mike Percy <mp...@apache.org> on 2014/07/23 02:41:03 UTC

Review Request 23830: Add simple regex search-and-replace interceptor

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

Review request for Flume.


Bugs: FLUME-2431
    https://issues.apache.org/jira/browse/FLUME-2431


Repository: flume-git


Description
-------

This patch implements a search-and-replace interceptor that can be used for simple generic inline transformations of event bodies.

At the time of writing, this interceptor assumes UTF-8 encoding of messages.


Diffs
-----

  flume-ng-core/src/main/java/org/apache/flume/interceptor/InterceptorType.java c84cea5 
  flume-ng-core/src/main/java/org/apache/flume/interceptor/SearchAndReplaceInterceptor.java PRE-CREATION 
  flume-ng-core/src/test/java/org/apache/flume/interceptor/TestSearchAndReplaceInterceptor.java PRE-CREATION 
  flume-ng-doc/sphinx/FlumeUserGuide.rst 1e98725 

Diff: https://reviews.apache.org/r/23830/diff/


Testing
-------

Added unit tests.


Thanks,

Mike Percy


Re: Review Request 23830: Add simple regex search-and-replace interceptor

Posted by Mike Percy <mp...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23830/
-----------------------------------------------------------

(Updated July 23, 2014, 1:29 a.m.)


Review request for Flume.


Changes
-------

Added an additional case to the unit test


Bugs: FLUME-2431
    https://issues.apache.org/jira/browse/FLUME-2431


Repository: flume-git


Description
-------

This patch implements a search-and-replace interceptor that can be used for simple generic inline transformations of event bodies.

At the time of writing, this interceptor assumes UTF-8 encoding of messages.


Diffs (updated)
-----

  flume-ng-configuration/src/main/java/org/apache/flume/Context.java 927636c 
  flume-ng-core/src/main/java/org/apache/flume/interceptor/InterceptorType.java c84cea5 
  flume-ng-core/src/main/java/org/apache/flume/interceptor/SearchAndReplaceInterceptor.java PRE-CREATION 
  flume-ng-core/src/test/java/org/apache/flume/interceptor/TestSearchAndReplaceInterceptor.java PRE-CREATION 
  flume-ng-doc/sphinx/FlumeUserGuide.rst 1e98725 

Diff: https://reviews.apache.org/r/23830/diff/


Testing
-------

Added unit tests.


Thanks,

Mike Percy


Re: Review Request 23830: Add simple regex search-and-replace interceptor

Posted by Mike Percy <mp...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23830/
-----------------------------------------------------------

(Updated July 23, 2014, 1:12 a.m.)


Review request for Flume.


Changes
-------

Added charset config support


Bugs: FLUME-2431
    https://issues.apache.org/jira/browse/FLUME-2431


Repository: flume-git


Description
-------

This patch implements a search-and-replace interceptor that can be used for simple generic inline transformations of event bodies.

At the time of writing, this interceptor assumes UTF-8 encoding of messages.


Diffs (updated)
-----

  flume-ng-configuration/src/main/java/org/apache/flume/Context.java 927636c 
  flume-ng-core/src/main/java/org/apache/flume/interceptor/InterceptorType.java c84cea5 
  flume-ng-core/src/main/java/org/apache/flume/interceptor/SearchAndReplaceInterceptor.java PRE-CREATION 
  flume-ng-core/src/test/java/org/apache/flume/interceptor/TestSearchAndReplaceInterceptor.java PRE-CREATION 
  flume-ng-doc/sphinx/FlumeUserGuide.rst 1e98725 

Diff: https://reviews.apache.org/r/23830/diff/


Testing
-------

Added unit tests.


Thanks,

Mike Percy