You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tez.apache.org by "László Bodor (Jira)" <ji...@apache.org> on 2020/04/27 11:40:00 UTC

[jira] [Commented] (TEZ-4157) Remove unused direct netty dependency

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

László Bodor commented on TEZ-4157:
-----------------------------------

after the patch:
{code}
mvn dependency:tree | tee dep.log
grep "netty" dep.log
{code}

results this below, not showing any '+' signs in first column (meaning that no direct dependency left)
{code}
[INFO] |  |  \- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  +- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  +- io.netty:netty-all:jar:4.0.52.Final:test
[INFO] |  |  \- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  |  \- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  \- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  \- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  +- io.netty:netty-all:jar:4.0.52.Final:test
[INFO] |  |  \- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |     \- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  +- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  +- io.netty:netty-all:jar:4.0.52.Final:test
[INFO] |  +- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  +- io.netty:netty-all:jar:4.0.52.Final:test
[INFO] |  |  \- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  +- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  +- io.netty:netty-all:jar:4.0.52.Final:test
[INFO] |  +- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  +- io.netty:netty-all:jar:4.0.52.Final:test
[INFO] |  |  |  \- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  +- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  +- io.netty:netty-all:jar:4.0.52.Final:test
[INFO] |  +- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  +- io.netty:netty-all:jar:4.0.52.Final:test
[INFO] |  +- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  \- io.netty:netty-all:jar:4.0.52.Final:provided
[INFO] |  +- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  +- io.netty:netty-all:jar:4.0.52.Final:test
[INFO] |  |  |  \- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |  |  \- io.netty:netty:jar:3.10.5.Final:compile
[INFO] |     \- io.netty:netty:jar:3.10.5.Final:compile
{code}

> Remove unused direct netty dependency
> -------------------------------------
>
>                 Key: TEZ-4157
>                 URL: https://issues.apache.org/jira/browse/TEZ-4157
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: László Bodor
>            Assignee: László Bodor
>            Priority: Major
>         Attachments: TEZ-4157.01.patch
>
>
> In the dependency tree, there are 2 occurrences of compile scope direct netty dependencies, however, they're not used at all. I compiled locally successfully without them.
> E.g. when investigating blackduck alerts (complaining about netty deps for current 3.10.5.Final), it would be cleaner to start from a dependency tree where Tez doesn't depend on netty directly in order to eliminate its responsibility (and move the focus to underlying hadoop for instance).



--
This message was sent by Atlassian Jira
(v8.3.4#803005)