You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "Emmanuel Bourg (JIRA)" <qp...@incubator.apache.org> on 2010/05/27 01:45:36 UTC
[jira] Commented: (QPID-2629) Remove Java client dependency on
commons-collection.jar
[ https://issues.apache.org/jira/browse/QPID-2629?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12872015#action_12872015 ]
Emmanuel Bourg commented on QPID-2629:
--------------------------------------
That's fine to copy the classes from Commons Collections, but you may want to change the package to avoid any conflict if Commons Collections is on the classpath.
Also this issue is for the Java client instead of the C++ client.
> Remove Java client dependency on commons-collection.jar
> -------------------------------------------------------
>
> Key: QPID-2629
> URL: https://issues.apache.org/jira/browse/QPID-2629
> Project: Qpid
> Issue Type: Improvement
> Components: C++ Client
> Reporter: Rajith Attapattu
> Assignee: Rajith Attapattu
> Priority: Minor
> Fix For: 0.7
>
> Attachments: QPID-2629.patch
>
>
> The JMS client has only a single dependency on the commons-collection.jar.
> It is using the RefereneMap in the message delegates to store the destination cache.
> It would be nice if we could use an alternative instead of having yet another dependency.
> For a client application it's really a big plus point to have as less dependencies as possible
> (Pls note that for the broker module, commons-collection is a transitive dependency, so we can't just remove it from our lib folder).
> But that is fine as it's only a compile time dependency should anybody wish to build the client from source.
> But for runtime we could definitely get rid of it to keep the size small.
> Possible strategies.
> 1. Replace the ReferenceMap with a suitable implementation. I don't think WeakHashMap as suggested on a thread is the right solution either.
> We currently use the ref map with hard refs for keys and soft refs for values.
> 2. We could just include the necessary files (just 4 actually) in the clients source dir. - This is perfectly legal as per the ASF license this is allowed.
> This also seems like a very good short term solution.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project: http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org