You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by "Hoss Man (JIRA)" <ji...@apache.org> on 2015/02/27 00:58:04 UTC
[jira] [Created] (SOLR-7171)
BaseDistributedSearchTestCase.getSolrXml() not used consistently - multiple
divergent ways a JettySolrRunner may be inited
Hoss Man created SOLR-7171:
------------------------------
Summary: BaseDistributedSearchTestCase.getSolrXml() not used consistently - multiple divergent ways a JettySolrRunner may be inited
Key: SOLR-7171
URL: https://issues.apache.org/jira/browse/SOLR-7171
Project: Solr
Issue Type: Bug
Reporter: Hoss Man
As part of SOLR-6349, i wanted to take advantage of the new features in SOLR-7147 to inspect shard requests in TestDistributedSearch, but even though i added a proper override of getSolrXml...
{code}
@Override
protected String getSolrXml() {
return "solr-trackingshardhandler.xml";
}
{code}
...that value was being ignored, and i was never getting an instance of TrackingShardHandlerFactory.
I poked around a bit and confirmed that getSolrXml() is used by "setupJettySolrHome(File)" but that method is never called by BaseDistributedSearchTestCase - it's only called in framework subclasses like AbstractDistribZkTestBase and AbstractFullDistribZkTestBase. Instead, for simple subclasses of BaseDistributedSearchTestCase the jetty instances seem to be coming from createServers(int)
----
I don't really understand why there are so many diff ways for a shard instance to be inited, and presumably that should be refactored? .. but a more immediate concern is that subclasses of BaseDistributedSearchTestCase which attempt to override the solr.xml file used can't (unless they are actually a subclass of AbstractDistribZkTestBase of AbstractFullDistribZkTestBase)
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org