accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher Tubbs (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-3579) Remove org.mortbay.jetty from pom
Date Wed, 11 Feb 2015 17:27:12 GMT


Christopher Tubbs commented on ACCUMULO-3579:

So, {{org.mortbay.jetty:jetty-util:jar:6.1.26}} is a dependency of {{hadoop-hdfs}}, which
is brought in by {{hadoop-client}}, and {{org.mortbay.jetty:jetty:jar:6.1.26}} is a dependency
of {{hadoop-common:test-jar}}, which is brought in via {{hadoop-minicluster}} for testing
and for {{accumulo-minicluster}}.

These will be brought in transitively, regardless of whether we specify it or not, so I'm
+1 to the patch, so long as we don't see any problems later as a result of the change. However,
I do not believe we are harmed by omitting them, and none of our code needs them. Ideally,
if these are not needed by Hadoop client apps and are only used for Hadoop-packaged services,
they should be specified as "optional" dependencies in upstream Hadoop, so they don't pollute
client applications which depend on Hadoop.

(Also, just noticed via {{grep -R mortbay .}} that some of our default logger configuration
and security policy template is mortbay-specific. Not sure if those are still needed for hadoop
client code, but probably not.)

> Remove org.mortbay.jetty from pom
> ---------------------------------
>                 Key: ACCUMULO-3579
>                 URL:
>             Project: Accumulo
>          Issue Type: Task
>          Components: build
>            Reporter: Josh Elser
>            Assignee: Josh Elser
>             Fix For: 1.7.0, 1.6.3
>         Attachments: 0001-ACCUMULO-3579-Remove-org.mortbay.jetty-from-poms.patch
> We have a dependency on org.mortbay.jetty:jetty:6.1.26 that appears to be used by nothing.

This message was sent by Atlassian JIRA

View raw message