You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-commits@lucene.apache.org by ry...@apache.org on 2008/10/28 17:45:33 UTC

svn commit: r708606 - in /lucene/solr/trunk: CHANGES.txt client/java/solrj/src/org/apache/solr/client/solrj/request/CoreAdminRequest.java

Author: ryan
Date: Tue Oct 28 09:45:32 2008
New Revision: 708606

URL: http://svn.apache.org/viewvc?rev=708606&view=rev
Log:
SOLR-827 -- fix solrj admin create core

Modified:
    lucene/solr/trunk/CHANGES.txt
    lucene/solr/trunk/client/java/solrj/src/org/apache/solr/client/solrj/request/CoreAdminRequest.java

Modified: lucene/solr/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/CHANGES.txt?rev=708606&r1=708605&r2=708606&view=diff
==============================================================================
--- lucene/solr/trunk/CHANGES.txt (original)
+++ lucene/solr/trunk/CHANGES.txt Tue Oct 28 09:45:32 2008
@@ -89,7 +89,7 @@
 
  4. SOLR-805: DisMax queries are not being cached in QueryResultCache (Todd Feak via koji)
 
- 5. SOLR-803: CoreAdminRequest.createCore fails because name parameter isn't set (Sean Colombo via ryan)
+ 5. SOLR-803, SOLR-827: Fix CoreAdminRequest#createCore (Sean Colombo via ryan)
  
 
 Other Changes

Modified: lucene/solr/trunk/client/java/solrj/src/org/apache/solr/client/solrj/request/CoreAdminRequest.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/client/java/solrj/src/org/apache/solr/client/solrj/request/CoreAdminRequest.java?rev=708606&r1=708605&r2=708606&view=diff
==============================================================================
--- lucene/solr/trunk/client/java/solrj/src/org/apache/solr/client/solrj/request/CoreAdminRequest.java (original)
+++ lucene/solr/trunk/client/java/solrj/src/org/apache/solr/client/solrj/request/CoreAdminRequest.java Tue Oct 28 09:45:32 2008
@@ -66,7 +66,11 @@
       }
       ModifiableSolrParams params = new ModifiableSolrParams();
       params.set( CoreAdminParams.ACTION, action.toString() );
-      params.set( CoreAdminParams.NAME, core );
+      if( action.equals(CoreAdminAction.CREATE) ) {
+        params.set( CoreAdminParams.NAME, core );
+      } else {
+        params.set( CoreAdminParams.CORE, core );
+      }
       params.set( CoreAdminParams.INSTANCE_DIR, instanceDir);
       if (configName != null) {
         params.set( CoreAdminParams.CONFIG, configName);