You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by mk...@apache.org on 2017/05/04 15:05:30 UTC
lucene-solr:master: SOLR-9867: fixing JvmMetricsTest broken earlier,
bring back testTechproductsExample() and single SDF.cores assignment.
Repository: lucene-solr
Updated Branches:
refs/heads/master 1534b6219 -> 5eb4a8d71
SOLR-9867: fixing JvmMetricsTest broken earlier, bring back testTechproductsExample()
and single SDF.cores assignment.
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/5eb4a8d7
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/5eb4a8d7
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/5eb4a8d7
Branch: refs/heads/master
Commit: 5eb4a8d71b491ca7f389b4dc7414dbc1a2620ae2
Parents: 1534b62
Author: Mikhail Khludnev <mk...@apache.org>
Authored: Thu May 4 17:18:24 2017 +0300
Committer: Mikhail Khludnev <mk...@apache.org>
Committed: Thu May 4 17:21:38 2017 +0300
----------------------------------------------------------------------
.../org/apache/solr/servlet/SolrDispatchFilter.java | 14 +++++++-------
.../org/apache/solr/util/TestSolrCLIRunExample.java | 3 +--
2 files changed, 8 insertions(+), 9 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/5eb4a8d7/solr/core/src/java/org/apache/solr/servlet/SolrDispatchFilter.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/servlet/SolrDispatchFilter.java b/solr/core/src/java/org/apache/solr/servlet/SolrDispatchFilter.java
index 02b0460..a4c9504 100644
--- a/solr/core/src/java/org/apache/solr/servlet/SolrDispatchFilter.java
+++ b/solr/core/src/java/org/apache/solr/servlet/SolrDispatchFilter.java
@@ -172,7 +172,7 @@ public class SolrDispatchFilter extends BaseSolrFilter {
coresInit = createCoreContainer(solrHome == null ? SolrResourceLoader.locateSolrHome() : Paths.get(solrHome),
extraProperties);
this.httpClient = coresInit.getUpdateShardHandler().getHttpClient();
- setupJvmMetrics();
+ setupJvmMetrics(coresInit);
log.debug("user.dir=" + System.getProperty("user.dir"));
}
catch( Throwable t ) {
@@ -190,9 +190,9 @@ public class SolrDispatchFilter extends BaseSolrFilter {
}
}
- private void setupJvmMetrics() {
- SolrMetricManager metricManager = cores.getMetricManager();
- final Set<String> hiddenSysProps = cores.getConfig().getHiddenSysProps();
+ private void setupJvmMetrics(CoreContainer coresInit) {
+ SolrMetricManager metricManager = coresInit.getMetricManager();
+ final Set<String> hiddenSysProps = coresInit.getConfig().getHiddenSysProps();
try {
String registry = SolrMetricManager.getRegistryName(SolrInfoBean.Group.jvm);
metricManager.registerAll(registry, new AltBufferPoolMetricSet(), true, "buffers");
@@ -247,9 +247,9 @@ public class SolrDispatchFilter extends BaseSolrFilter {
*/
protected CoreContainer createCoreContainer(Path solrHome, Properties extraProperties) {
NodeConfig nodeConfig = loadNodeConfig(solrHome, extraProperties);
- cores = new CoreContainer(nodeConfig, extraProperties, true);
- cores.load();
- return cores;
+ final CoreContainer coreContainer = new CoreContainer(nodeConfig, extraProperties, true);
+ coreContainer.load();
+ return coreContainer;
}
/**
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/5eb4a8d7/solr/core/src/test/org/apache/solr/util/TestSolrCLIRunExample.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/util/TestSolrCLIRunExample.java b/solr/core/src/test/org/apache/solr/util/TestSolrCLIRunExample.java
index 86790c8..6bce8226 100644
--- a/solr/core/src/test/org/apache/solr/util/TestSolrCLIRunExample.java
+++ b/solr/core/src/test/org/apache/solr/util/TestSolrCLIRunExample.java
@@ -50,7 +50,6 @@ import org.apache.solr.common.SolrInputDocument;
import org.junit.After;
import org.junit.AfterClass;
import org.junit.BeforeClass;
-import org.junit.Ignore;
import org.junit.Test;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -297,7 +296,7 @@ public class TestSolrCLIRunExample extends SolrTestCaseJ4 {
closeables = null;
}
}
- @Ignore
+
@Test
public void testTechproductsExample() throws Exception {
testExample("techproducts");