[ 
https://issues.apache.org/jira/browse/FLUME-940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13197609#comment-13197609
 ] 

[email protected] commented on FLUME-940:
-----------------------------------------------------


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

Ship it!


lgtm

- Prasad


On 2012-01-30 22:37:38, Arvind Prabhakar wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/3698/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-01-30 22:37:38)
bq.  
bq.  
bq.  Review request for Flume and Prasad Mujumdar.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Removed the FanoutChannel and JsonFileConfigurationProvider implementation.
bq.  
bq.  
bq.  This addresses bug FLUME-940.
bq.      https://issues.apache.org/jira/browse/FLUME-940
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    flume-ng-core/src/main/java/org/apache/flume/channel/ChannelType.java 
b9ce7a5 
bq.    flume-ng-core/src/main/java/org/apache/flume/channel/FanoutChannel.java 
ca21794 
bq.    
flume-ng-core/src/test/java/org/apache/flume/channel/TestFanoutChannel.java 
ada9a72 
bq.    
flume-ng-node/src/main/java/org/apache/flume/conf/file/JsonFileConfigurationProvider.java
 f48e681 
bq.    
flume-ng-node/src/test/java/org/apache/flume/conf/file/TestJsonFileConfigurationProvider.java
 ce4e752 
bq.    
flume-ng-node/src/test/java/org/apache/flume/conf/properties/TestPropertiesFileConfigurationProvider.java
 2b1599d 
bq.  
bq.  Diff: https://reviews.apache.org/r/3698/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Ran all unit tests. 
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Arvind
bq.  
bq.


                
> Remove unused code from Flume
> -----------------------------
>
>                 Key: FLUME-940
>                 URL: https://issues.apache.org/jira/browse/FLUME-940
>             Project: Flume
>          Issue Type: Task
>    Affects Versions: v1.0.0
>            Reporter: Arvind Prabhakar
>            Assignee: Arvind Prabhakar
>             Fix For: v1.1.0
>
>         Attachments: FLUME-940-1-svn.diff
>
>
> This includes the FanoutChannel as well as JsonFileConfigurationProvider. 
> Since we have these in the history of source control, we can revive them at 
> anytime as needed rather than incrementally maintaining and not use them 
> going forward.

--
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

        

Reply via email to