Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 337184 - Bug in configuring a new collection for a SolrServerRepository with SolrServerType.REMOTE
Summary: Bug in configuring a new collection for a SolrServerRepository with SolrServe...
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: gyrex (show other bugs)
Version: unspecified   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Mike Tschierschke CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-15 03:11 EST by Mike Tschierschke CLA
Modified: 2018-03-19 11:59 EDT (History)
2 users (show)

See Also:

Attachments
resolves the bug with new API for ISolrRepositoryConfigurer (4.27 KB, patch)
2011-02-15 08:15 EST, Mike Tschierschke CLA 		gunnar: review+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Tschierschke CLA 2011-02-15 03:11:07 EST 
It's not possible at the moment to configure a remote server url for a new collection in solr repositories. Seems that we need a rework on the ISolrRepositoryConfigurer API to fix this.
Comment 1 Mike Tschierschke CLA 2011-02-15 08:15:13 EST 
Created attachment 188990 [details]
resolves the bug with new API for ISolrRepositoryConfigurer
Comment 2 Mike Tschierschke CLA 2011-02-15 08:24:34 EST 
The attached patch fixes this bug through little changes on ISolrRepositoryConfigurer API and impl.

My Tests with a local configuration passed. After configuring a remote solr server in gyrex my indices where created and read successfully. The test server was the example solr server you'll found here: http://apache.openmirror.de//lucene/solr/1.4.1/apache-solr-1.4.1.zip

Gunnar, please review my changes, resolve the bug if you want to and let us talk about the next steps.
Comment 3 Gunnar Wagenknecht CLA 2011-02-15 15:21:21 EST 
Comment on attachment 188990 [details]
resolves the bug with new API for ISolrRepositoryConfigurer

Patch looks good. Please go ahead and commit. Thanks Mike!
Comment 4 Mike Tschierschke CLA 2011-02-17 04:32:44 EST 
fixed with API rework on ISolrRepositoryConfigurer and impl. Removed: void addCollection(String, SolrServerType) - Added void addCollection(String) for embedded server, void addCollection(String, SolrServerType, String) for any other server type