[ https://issues.apache.org/jira/browse/SOLR-3344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13250753#comment-13250753
]
Steven Rowe commented on SOLR-3344:
-----------------------------------
bq. I like the optional dependencies alternative better: add optional jetty dependencies to
Solr test-framework, and change the javadoc for BaseDistributedSearchTestCase to note this
requirement.
Thinking about this more, the rationale I would ordinarily use to justify making a dependency
optional involves minimizing *runtime* dependencies. But the Solr test-framework is a test-only
module, and requiring *test* dependencies that won't be used by all consumers should not cause
any undue hardship.
So I've changed my mind. I'll put up a superset of your patch, Benson, that removes the dataimporthandler
and clustering POMs' jetty dependencies. I'll leave solr-core's optional dependency as-is.
> POM dependencies not all there yet for solr-test-framework
> ----------------------------------------------------------
>
> Key: SOLR-3344
> URL: https://issues.apache.org/jira/browse/SOLR-3344
> Project: Solr
> Issue Type: Bug
> Components: Build
> Affects Versions: 4.0
> Reporter: Benson Margulies
> Attachments: 0001-Add-jetty-deps.patch
>
>
> The pom for solr-test-framework does not mention jetty, so a test case that doesn't get
jetty into it's classpath otherwise fails with:
> {noformat}
> java.lang.NoClassDefFoundError: org/eclipse/jetty/server/SessionIdManager
> {noformat}
--
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
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org
|