[
https://issues.apache.org/jira/browse/SOLR-1224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12720445#action_12720445
]
Ryan McKinley commented on SOLR-1224:
-------------------------------------
For the record... 1.3 had three jars with a strange circular dependency graph.
* solr-common (self sufficient)
* solr-solrj (requires solr-common and parts of solr-core)
* solr-core (requires solr-common and parts of solr-solrj)
In 1.4 there are only two jars:
* solr-solrj (solr-common+solr-j)
* solr-core (requires solr-solrj)
perhaps the names should change, but it is more a legacy issue now.
> NamedList contained in SolrJ jar file
> -------------------------------------
>
> Key: SOLR-1224
> URL: https://issues.apache.org/jira/browse/SOLR-1224
> Project: Solr
> Issue Type: Bug
> Components: clients - java
> Affects Versions: 1.4
> Environment: Solr 1.4, Java 1.5.0_18, Jetty 6.1.18, Windows XP SP3
> Reporter: Daryl Beattie
>
> In Solr 1.3, NamedList (for example, that being returned by
> SolrInfoMBean.getStatistics()) was in org.apache.solr.util, but it has
> changed to be NamedList in org.apache.solr.common.util (looks like it changed
> packages). Unfortunately, the class file has also moved over to the SolrJ JAR
> file, probably because people using SolrJ need access to it.
> I would assert that the NamedList class file should not be in the SolrJ JAR
> file, because people writing code against Solr-core shouldn't be required to
> include the SolrJ JAR in their projects just so that they can access the
> NamedList object (for example, when reading stats from the SolrInfoMBean),
> and thus it should be moved back to the solr-core jar file.
> (Oddly, the original NamedList, now deprecated, is still present in
> org.apache.solr.util in the solr-core JAR, which is a related problem. But
> code has been rewritten to return the new NamedList in
> org.apache.solr.common.util.)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
