You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flume.apache.org by Eric Sammer <es...@cloudera.com> on 2011/12/29 23:16:22 UTC

Review Request: FLUME-913: slf4j-log4j binding is excluded from the dist assembly due to test scope

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

Review request for Flume.


Summary
-------

Removed test scope from slf4j-log4j dep in flume-ng-core.


This addresses bug FLUME-913.
    https://issues.apache.org/jira/browse/FLUME-913


Diffs
-----

  flume-ng-core/pom.xml e4fb829 

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


Testing
-------


Thanks,

Eric


Re: Review Request: FLUME-913: slf4j-log4j binding is excluded from the dist assembly due to test scope

Posted by Arvind Prabhakar <ar...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3333/#review4157
-----------------------------------------------------------

Ship it!


+1

- Arvind


On 2011-12-29 22:16:22, Eric Sammer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/3333/
> -----------------------------------------------------------
> 
> (Updated 2011-12-29 22:16:22)
> 
> 
> Review request for Flume.
> 
> 
> Summary
> -------
> 
> Removed test scope from slf4j-log4j dep in flume-ng-core.
> 
> 
> This addresses bug FLUME-913.
>     https://issues.apache.org/jira/browse/FLUME-913
> 
> 
> Diffs
> -----
> 
>   flume-ng-core/pom.xml e4fb829 
> 
> Diff: https://reviews.apache.org/r/3333/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Eric
> 
>