Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: jira@apache.org
Date: Wed, 11 Feb 2015 17:27:12 +0000 (UTC)
From: "Christopher Tubbs (JIRA)" <jira@apache.org>
To: notifications@accumulo.apache.org
Message-ID: <JIRA.12774075.1423625392000.15520.1423675632436@Atlassian.JIRA>
In-Reply-To: <JIRA.12774075.1423625392000@Atlassian.JIRA>
References: <JIRA.12774075.1423625392000@Atlassian.JIRA>
 <JIRA.12774075.1423625392317@arcas>
Subject: [jira] [Commented] (ACCUMULO-3579) Remove org.mortbay.jetty from
 pom
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

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: https://issues.apache.org/jira/browse/ACCUMULO-3579
>             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
(v6.3.4#6332)