Thanks Martin, it is a very nice summary !

Here my small comments :

I'd like to suggest the following:

1. Introduce the index store (as Christophe already done).
The index store has on principle the methods
- index (uri, nodeRevisionDescriptor, nodeRevisionContent)
- drop (uri, nodeRevisionDescriptor)
They are called at create, update, delete. No search method!


Agree, It was a bad idea to introduce the search method in this helper class and in the index store.

In Domain.xml we could have something like:

<store name="jdbc" classname="org.apache.slide.store.BindingStore">
 <nodestore classname="org.apache.slide.store.impl.rdbms.JDBCStore">
   <parameter name="driver">com.mysql.jdbc.Driver</parameter>
   <parameter 
name="url">jdbc:mysql://localhost:3306/test?autoReconnect=true</parameter>
   <parameter name="user">wam</parameter>
   <parameter name="password"/>
   <parameter 
name="adapter">org.apache.slide.store.impl.rdbms.MySqlRDBMSAdapter</parameter>
 </nodestore>
 <securitystore>
   <reference store="nodestore"/>
 </securitystore>
 <lockstore>
   <reference store="nodestore"/>
 </lockstore>
 <revisiondescriptorsstore>
   <reference store="nodestore"/>
 </revisiondescriptorsstore>
 <revisiondescriptorstore>
   <reference store="nodestore"/>
 </revisiondescriptorstore>
 <contentstore classname="org.apache.slide.store.txfile.TxFileContentStore">
   <parameter name="rootpath">mysql/store/content</parameter>
   <parameter name="workpath">mysql/work/content</parameter>
 </contentstore>
 <indexer classname="my.lucene.Indexer"/>
 <searchengine>
   <parameter name="propertySearchClass">my.sql.SearchEngine</parameter>
   <parameter name="ContentSearchClass">my.Lucene.SearchEngine</parameter>
 </searchengine>
</store>

Does this make sense?



Good. So, let's start with that no ?
Indexer is almost done, I can clean-up my code, make a default implementation based on Lucene and send another cvs diff.
For the "searchengine", I will be happy to help if needed.



Christophe




---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to