good idea.
The first set of methods are fine. Returning a modifiable collection
is not 'good design'
 void registerCommitCallback( SolrEventListener listener);
 void registerOptimizeCallback( SolrEventListener listener);

Similarly we must also add
void registerQueryResponseWriter(String wt, QueryResponseWriter writer)
to SolrCore.

--Noble



On Thu, Jun 26, 2008 at 8:13 AM, Ryan McKinley <[EMAIL PROTECTED]> wrote:
> How do you all feel about exposing access to the commit/optimize callbacks
> in UpdateHandler?  Perhaps adding the functions:
>  void registerCommitCallback( SolrEventListener );
>  void registerCommitCallback( SolrEventListener );
> and/or maybe:
>  Collection<SolrEventListener> getCommitCollbacks();
>  Collection<SolrEventListener> getOptimizeCollbacks();
>
> if the returned collection is modifiable, we could directly add to it.
>
> Currently everything needs to be registered via solrconfig.xml -- but I
> would like an UpdateProcessor to be a commit/optimize callback.
>
> Someday we could do this with spring...



-- 
--Noble Paul

Reply via email to