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

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


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

Review request for Flume and Prasad Mujumdar.


Summary
-------

* Remove Hadoop as a flume-ng-core dep.
* Added commons-cli as an explicit dep. We were accidentally getting it via 
hadoop.
* Cleaned up unused org.apache.hadoop.* class usage in the ExecSource test.


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


Diffs
-----

  i/flume-ng-core/pom.xml 90dbcaa 
  i/flume-ng-core/src/test/java/org/apache/flume/source/TestExecSource.java 
6607bc1 

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


Testing
-------

Unit tests pass.


Thanks,

Eric


                
> Need to remove dependency on hadoop core from flume-ng-core project
> -------------------------------------------------------------------
>
>                 Key: FLUME-825
>                 URL: https://issues.apache.org/jira/browse/FLUME-825
>             Project: Flume
>          Issue Type: Bug
>          Components: Build
>    Affects Versions: NG alpha 1
>            Reporter: Will McQueen
>             Fix For: NG alpha 2
>
>
> flume-ng-core's pom.xml file says:
> FIXME: There should be no dependency on Hadoop in Flume core.
> Please remove the dependency. Thanks.

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