Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 337404

Summary: [cds] Rework of the SolrRepository concept and impl
Product: z_Archived Reporter: Mike Tschierschke <mike.tschierschke>
Component: gyrexAssignee: Mike Tschierschke <mike.tschierschke>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: andreas.mihm, gunnar
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
removed solr multiple collections capability
none
simplified IDocumentManager none

Description Mike Tschierschke CLA 2011-02-17 04:55:15 EST 
At the moment each Gyrex application, that uses solr indices, has max one solr repository in each runtime context. Different indices are modeled as different collections in each repository.

This concept is obsolete. Each solr repository should represent one index. This relation must not be one-to-one. Different repos may connect to the same index.
Comment 1 Gunnar Wagenknecht CLA 2011-02-19 04:06:19 EST 
As discussed, we should rework the concept and remove collections all together. They aren't bad per definition but they add a layer of complexity that we don't want to introduce at this stage.

Mike, you also need to refactor the Solr based CDS managers to allow extensibility by others. Once you have a patch ready feel free to attach for API discussion.
Comment 2 Mike Tschierschke CLA 2011-02-24 02:05:11 EST 
Created attachment 189672 [details]
removed solr multiple collections capability

Finished the rework on solr repositories by removing the capability to manage multiple document collections. Now, each solr repository is a wrapper for exactly one collection.
Also removed pre-definded repository content types for document and facet repos.

Gunnar, please review my changes.
Comment 3 Mike Tschierschke CLA 2011-02-25 09:51:47 EST 
Created attachment 189807 [details]
simplified IDocumentManager

As discussed, I simplified the IDocumentManager concept by removing the interfaces IDocumentCollection, ISolrDocumentCollection and ISolrQueryExecutor. Each method is now integrated in the IDocumentManager and impl
Comment 4 Gunnar Wagenknecht CLA 2011-02-26 11:19:13 EST 
Patch committed. Thanks Mike!