You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ro...@apache.org on 2013/12/18 18:39:16 UTC
svn commit: r1552045 - in /lucene/dev/branches/branch_4x: ./ solr/
solr/core/ solr/core/src/java/org/apache/solr/core/
solr/core/src/test-files/solr/ solr/core/src/test/org/apache/solr/core/
Author: romseygeek
Date: Wed Dec 18 17:39:15 2013
New Revision: 1552045
URL: http://svn.apache.org/r1552045
Log:
SOLR-5556: Allow CollectionsHandler and InfoHandler classes to be specified in solr.xml
Modified:
lucene/dev/branches/branch_4x/ (props changed)
lucene/dev/branches/branch_4x/solr/ (props changed)
lucene/dev/branches/branch_4x/solr/CHANGES.txt (contents, props changed)
lucene/dev/branches/branch_4x/solr/core/ (props changed)
lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolr.java
lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolrXml.java
lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolrXmlOld.java
lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/CoreContainer.java
lucene/dev/branches/branch_4x/solr/core/src/test-files/solr/solr-50-all.xml
lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/core/TestCoreContainer.java
lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/core/TestSolrXml.java
Modified: lucene/dev/branches/branch_4x/solr/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_4x/solr/CHANGES.txt?rev=1552045&r1=1552044&r2=1552045&view=diff
==============================================================================
--- lucene/dev/branches/branch_4x/solr/CHANGES.txt (original)
+++ lucene/dev/branches/branch_4x/solr/CHANGES.txt Wed Dec 18 17:39:15 2013
@@ -93,6 +93,9 @@ New Features
* SOLR-5522: Remove EditFileRequestHandler from 4x (Erick Erickson)
+* SOLR-5556: Allow class of CollectionsHandler and InfoHandler to be specified
+ in solr.xml. (Gregory Chanan, Alan Woodward)
+
Bug Fixes
----------------------
Modified: lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolr.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolr.java?rev=1552045&r1=1552044&r2=1552045&view=diff
==============================================================================
--- lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolr.java (original)
+++ lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolr.java Wed Dec 18 17:39:15 2013
@@ -191,6 +191,14 @@ public abstract class ConfigSolr {
return get(CfgProp.SOLR_ADMINHANDLER, "org.apache.solr.handler.admin.CoreAdminHandler");
}
+ public String getCollectionsHandlerClass() {
+ return get(CfgProp.SOLR_COLLECTIONSHANDLER, "org.apache.solr.handler.admin.CollectionsHandler");
+ }
+
+ public String getInfoHandlerClass() {
+ return get(CfgProp.SOLR_INFOHANDLER, "org.apache.solr.handler.admin.InfoHandler");
+ }
+
public boolean hasSchemaCache() {
return getBool(ConfigSolr.CfgProp.SOLR_SHARESCHEMA, false);
}
@@ -215,6 +223,7 @@ public abstract class ConfigSolr {
// Ugly for now, but we'll at least be able to centralize all of the differences between 4x and 5x.
protected static enum CfgProp {
SOLR_ADMINHANDLER,
+ SOLR_COLLECTIONSHANDLER,
SOLR_CORELOADTHREADS,
SOLR_COREROOTDIRECTORY,
SOLR_DISTRIBUPDATECONNTIMEOUT,
@@ -224,6 +233,7 @@ public abstract class ConfigSolr {
SOLR_HOST,
SOLR_HOSTCONTEXT,
SOLR_HOSTPORT,
+ SOLR_INFOHANDLER,
SOLR_LEADERVOTEWAIT,
SOLR_LOGGING_CLASS,
SOLR_LOGGING_ENABLED,
Modified: lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolrXml.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolrXml.java?rev=1552045&r1=1552044&r2=1552045&view=diff
==============================================================================
--- lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolrXml.java (original)
+++ lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolrXml.java Wed Dec 18 17:39:15 2013
@@ -103,6 +103,8 @@ public class ConfigSolrXml extends Confi
private void fillPropMap() {
propMap.put(CfgProp.SOLR_ADMINHANDLER, doSub("solr/str[@name='adminHandler']"));
+ propMap.put(CfgProp.SOLR_COLLECTIONSHANDLER, doSub("solr/str[@name='collectionsHandler']"));
+ propMap.put(CfgProp.SOLR_INFOHANDLER, doSub("solr/str[@name='infoHandler']"));
propMap.put(CfgProp.SOLR_CORELOADTHREADS, doSub("solr/int[@name='coreLoadThreads']"));
propMap.put(CfgProp.SOLR_COREROOTDIRECTORY, doSub("solr/str[@name='coreRootDirectory']"));
propMap.put(CfgProp.SOLR_DISTRIBUPDATECONNTIMEOUT, doSub("solr/solrcloud/int[@name='distribUpdateConnTimeout']"));
Modified: lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolrXmlOld.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolrXmlOld.java?rev=1552045&r1=1552044&r2=1552045&view=diff
==============================================================================
--- lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolrXmlOld.java (original)
+++ lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/ConfigSolrXmlOld.java Wed Dec 18 17:39:15 2013
@@ -139,6 +139,8 @@ public class ConfigSolrXmlOld extends Co
propMap.put(CfgProp.SOLR_ADMINHANDLER,
config.getVal("solr/cores/@adminHandler", false));
+ propMap.put(CfgProp.SOLR_COLLECTIONSHANDLER, config.getVal("solr/cores/@collectionsHandler", false));
+ propMap.put(CfgProp.SOLR_INFOHANDLER, config.getVal("solr/cores/@infoHandler", false));
propMap.put(CfgProp.SOLR_DISTRIBUPDATECONNTIMEOUT,
config.getVal("solr/cores/@distribUpdateConnTimeout", false));
propMap.put(CfgProp.SOLR_DISTRIBUPDATESOTIMEOUT,
Modified: lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/CoreContainer.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/CoreContainer.java?rev=1552045&r1=1552044&r2=1552045&view=diff
==============================================================================
--- lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/CoreContainer.java (original)
+++ lucene/dev/branches/branch_4x/solr/core/src/java/org/apache/solr/core/CoreContainer.java Wed Dec 18 17:39:15 2013
@@ -214,9 +214,9 @@ public class CoreContainer {
zkSys.initZooKeeper(this, solrHome, cfg);
- collectionsHandler = new CollectionsHandler(this);
- infoHandler = new InfoHandler(this);
- coreAdminHandler = createMultiCoreHandler(cfg.getCoreAdminHandlerClass());
+ collectionsHandler = createHandler(cfg.getCollectionsHandlerClass(), CollectionsHandler.class);
+ infoHandler = createHandler(cfg.getInfoHandlerClass(), InfoHandler.class);
+ coreAdminHandler = createHandler(cfg.getCoreAdminHandlerClass(), CoreAdminHandler.class);
containerProperties = cfg.getSolrProperties("solr");
@@ -778,10 +778,10 @@ public class CoreContainer {
return null;
}
- /**
+ /**
* Gets a core by name and increase its refcount.
*
- * @see SolrCore#close()
+ * @see SolrCore#close()
* @param name the core name
* @return the core if found, null if a SolrCore by this name does not exist
* @exception SolrException if a SolrCore with this name failed to be initialized
@@ -800,7 +800,7 @@ public class CoreContainer {
// OK, it's not presently in any list, is it in the list of dynamic cores but not loaded yet? If so, load it.
CoreDescriptor desc = solrCores.getDynamicDescriptor(name);
if (desc == null) { //Nope, no transient core with this name
-
+
// if there was an error initalizing this core, throw a 500
// error with the details for clients attempting to access it.
Exception e = getCoreInitFailures().get(name);
@@ -831,7 +831,7 @@ public class CoreContainer {
}
} catch(Exception ex){
// remains to be seen how transient cores and such
- // will work in SolrCloud mode, but just to be future
+ // will work in SolrCloud mode, but just to be future
// proof...
/*if (isZooKeeperAware()) {
try {
@@ -851,34 +851,33 @@ public class CoreContainer {
return core;
}
- // ---------------- Multicore self related methods ---------------
- /**
- * Creates a CoreAdminHandler for this MultiCore.
- * @return a CoreAdminHandler
- */
- protected CoreAdminHandler createMultiCoreHandler(final String adminHandlerClass) {
- return loader.newAdminHandlerInstance(CoreContainer.this, adminHandlerClass);
+ // ---------------- CoreContainer request handlers --------------
+
+ protected <T> T createHandler(String handlerClass, Class<T> clazz) {
+ return loader.newInstance(handlerClass, clazz, null, new Class[] { CoreContainer.class }, new Object[] { this });
}
public CoreAdminHandler getMultiCoreHandler() {
return coreAdminHandler;
}
-
+
public CollectionsHandler getCollectionsHandler() {
return collectionsHandler;
}
-
+
public InfoHandler getInfoHandler() {
return infoHandler;
}
+ // ---------------- Multicore self related methods ---------------
+
/**
* the default core name, or null if there is no default core name
*/
public String getDefaultCoreName() {
return cfg.getDefaultCoreName();
}
-
+
// all of the following properties aren't synchronized
// but this should be OK since they normally won't be changed rapidly
@Deprecated
Modified: lucene/dev/branches/branch_4x/solr/core/src/test-files/solr/solr-50-all.xml
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_4x/solr/core/src/test-files/solr/solr-50-all.xml?rev=1552045&r1=1552044&r2=1552045&view=diff
==============================================================================
--- lucene/dev/branches/branch_4x/solr/core/src/test-files/solr/solr-50-all.xml (original)
+++ lucene/dev/branches/branch_4x/solr/core/src/test-files/solr/solr-50-all.xml Wed Dec 18 17:39:15 2013
@@ -17,8 +17,10 @@
<solr>
<str name="adminHandler">testAdminHandler</str>
+ <str name="collectionsHandler">testCollectionsHandler</str>
<int name="coreLoadThreads">11</int>
<str name="coreRootDirectory">${coreRootDirectory:testCoreRootDirectory}</str>
+ <str name="infoHandler">testInfoHandler</str>
<str name="managementPath">testManagementPath</str>
<str name="sharedLib">testSharedLib</str>
<str name="shareSchema">${shareSchema:testShareSchema}</str>
Modified: lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/core/TestCoreContainer.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/core/TestCoreContainer.java?rev=1552045&r1=1552044&r2=1552045&view=diff
==============================================================================
--- lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/core/TestCoreContainer.java (original)
+++ lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/core/TestCoreContainer.java Wed Dec 18 17:39:15 2013
@@ -21,6 +21,9 @@ import org.apache.commons.io.FileUtils;
import org.apache.lucene.util.IOUtils;
import org.apache.lucene.util._TestUtil;
import org.apache.solr.SolrTestCaseJ4;
+import org.apache.solr.handler.admin.CollectionsHandler;
+import org.apache.solr.handler.admin.CoreAdminHandler;
+import org.apache.solr.handler.admin.InfoHandler;
import org.junit.AfterClass;
import org.junit.BeforeClass;
import org.junit.Test;
@@ -39,6 +42,9 @@ import java.util.jar.JarOutputStream;
import org.apache.lucene.util.Constants;
+import static org.hamcrest.core.Is.is;
+import static org.hamcrest.core.IsInstanceOf.instanceOf;
+
public class TestCoreContainer extends SolrTestCaseJ4 {
private static String oldSolrHome;
@@ -290,4 +296,48 @@ public class TestCoreContainer extends S
private static final String EMPTY_SOLR_XML2 ="<?xml version=\"1.0\" encoding=\"UTF-8\" ?>\n" +
"<solr>\n" +
"</solr>";
+
+ private static final String CUSTOM_HANDLERS_SOLR_XML = "<?xml version=\"1.0\" encoding=\"UTF-8\" ?>\n" +
+ "<solr>" +
+ " <str name=\"collectionsHandler\">" + CustomCollectionsHandler.class.getName() + "</str>" +
+ " <str name=\"infoHandler\">" + CustomInfoHandler.class.getName() + "</str>" +
+ " <str name=\"adminHandler\">" + CustomCoreAdminHandler.class.getName() + "</str>" +
+ "</solr>";
+
+ public static class CustomCollectionsHandler extends CollectionsHandler {
+ public CustomCollectionsHandler(CoreContainer cc) {
+ super(cc);
+ }
+ }
+
+ public static class CustomInfoHandler extends InfoHandler {
+ public CustomInfoHandler(CoreContainer cc) {
+ super(cc);
+ }
+ }
+
+ public static class CustomCoreAdminHandler extends CoreAdminHandler {
+ public CustomCoreAdminHandler(CoreContainer cc) {
+ super(cc);
+ }
+ }
+
+ @Test
+ public void testCustomHandlers() throws Exception {
+
+ ConfigSolr config = ConfigSolr.fromString(CUSTOM_HANDLERS_SOLR_XML);
+ SolrResourceLoader loader = new SolrResourceLoader("solr/collection1");
+
+ CoreContainer cc = new CoreContainer(loader, config);
+ try {
+ cc.load();
+ assertThat(cc.getCollectionsHandler(), is(instanceOf(CustomCollectionsHandler.class)));
+ assertThat(cc.getInfoHandler(), is(instanceOf(CustomInfoHandler.class)));
+ assertThat(cc.getMultiCoreHandler(), is(instanceOf(CustomCoreAdminHandler.class)));
+ }
+ finally {
+ cc.shutdown();
+ }
+
+ }
}
Modified: lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/core/TestSolrXml.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/core/TestSolrXml.java?rev=1552045&r1=1552044&r2=1552045&view=diff
==============================================================================
--- lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/core/TestSolrXml.java (original)
+++ lucene/dev/branches/branch_4x/solr/core/src/test/org/apache/solr/core/TestSolrXml.java Wed Dec 18 17:39:15 2013
@@ -47,6 +47,8 @@ public class TestSolrXml extends SolrTes
ConfigSolr cfg = ConfigSolr.fromSolrHome(loader, solrHome.getAbsolutePath());
assertEquals("Did not find expected value", cfg.get(ConfigSolr.CfgProp.SOLR_ADMINHANDLER, null), "testAdminHandler");
+ assertEquals("Did not find expected value", cfg.get(ConfigSolr.CfgProp.SOLR_COLLECTIONSHANDLER, null), "testCollectionsHandler");
+ assertEquals("Did not find expected value", cfg.get(ConfigSolr.CfgProp.SOLR_INFOHANDLER, null), "testInfoHandler");
assertEquals("Did not find expected value", cfg.getInt(ConfigSolr.CfgProp.SOLR_CORELOADTHREADS, 0), 11);
assertEquals("Did not find expected value", cfg.get(ConfigSolr.CfgProp.SOLR_COREROOTDIRECTORY, null), "testCoreRootDirectory");
assertEquals("Did not find expected value", cfg.getInt(ConfigSolr.CfgProp.SOLR_DISTRIBUPDATECONNTIMEOUT, 0), 22);