[
https://issues.apache.org/jira/browse/FLUME-1043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13235458#comment-13235458
]
[email protected] commented on FLUME-1043:
------------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4446/
-----------------------------------------------------------
Review request for Flume.
Summary
-------
Simple patch to use optional instead of compile scope for deps used in unit
tests in the SDK.
We can't use test scope because it transitively affects core which causes the
jars to not be included in the overall assembly. Optional seems to work fine.
This addresses bug FLUME-1043.
https://issues.apache.org/jira/browse/FLUME-1043
Diffs
-----
flume-ng-sdk/pom.xml c8fb582
Diff: https://reviews.apache.org/r/4446/diff
Testing
-------
Unit tests pass, able to run binary build with sample config.
Thanks,
Mike
> SDK should mark slf4j deps as optional
> --------------------------------------
>
> Key: FLUME-1043
> URL: https://issues.apache.org/jira/browse/FLUME-1043
> Project: Flume
> Issue Type: Improvement
> Affects Versions: v1.1.0
> Reporter: Mike Percy
> Assignee: Mike Percy
> Priority: Minor
> Fix For: v1.2.0
>
>
> The flume SDK marks the slf4j deps as normal deps due to test scope
> transitively affecting flume core. However a better approach is to use
> optional, which has the effect we want (even though the semantics are a bit
> off).
--
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
