[ 
https://issues.apache.org/jira/browse/SOLR-2193?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13041776#comment-13041776
 ] 

Robert Muir commented on SOLR-2193:
-----------------------------------

bq. Robert I fully agree, however then the title of the Jira is incorrect. 

How is it incorrect? The description of the issue is to overhaul the 
UpdateHandler, not to rearchitect Solr for NRT, move IW into SolrCore, or other 
things.

bq. We're also adding ref counting on IWs now as well? All of this is 
unnecessary. If we're modularizing, this isn't right path to go one.

How do you propose supporting core reloads without it? It seems to me (again 
solr dummy) that some mechanism like this is needed.

I guess personally, just looking at the big picture, this issue seems like a 
great win for Solr.

I tested the patch and all tests pass... if there are technical concerns with 
test coverage I'm happy to try to help there to help see this thing committed.


> Re-architect Update Handler
> ---------------------------
>
>                 Key: SOLR-2193
>                 URL: https://issues.apache.org/jira/browse/SOLR-2193
>             Project: Solr
>          Issue Type: Improvement
>            Reporter: Mark Miller
>             Fix For: 4.0
>
>         Attachments: SOLR-2193.patch, SOLR-2193.patch, SOLR-2193.patch, 
> SOLR-2193.patch, SOLR-2193.patch, SOLR-2193.patch
>
>
> The update handler needs an overhaul.
> A few goals I think we might want to look at:
> 1. Cleanup - drop DirectUpdateHandler(2) line - move to something like 
> UpdateHandler, DefaultUpdateHandler
> 2. Expose the SolrIndexWriter in the api or add the proper abstractions to 
> get done what we now do with special casing:
> if (directupdatehandler2)
>   success
>  else
>   failish
> 3. Stop closing the IndexWriter and start using commit (still lazy IW init 
> though).
> 4. Drop iwAccess, iwCommit locks and sync mostly at the Lucene level.
> 5. Keep NRT support in mind.
> 6. Keep microsharding in mind (maintain logical index as multiple physical 
> indexes)
> 7. Address the current issues we face because multiple original/'reloaded' 
> cores can have a different IndexWriter on the same index.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to