It turns out I'd already done this in some (uncommitted) code for SOLR-4196, I'll see what I can do there.
On Tue, Jan 29, 2013 at 2:49 PM, Yonik Seeley (JIRA) <[email protected]> wrote: > > [ > https://issues.apache.org/jira/browse/SOLR-4300?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13565715#comment-13565715 > ] > > Yonik Seeley commented on SOLR-4300: > ------------------------------------ > > I haven't had a chance to check out the whole patch in context... but if we > get a chance to revisit at any point, it would probably be nicer to use a > mechanism other than sleep/poll. Either wait/notify or Future would seem to > fit the bill. > >> Possible race condition in CoreContainer.getCore() when lazily loading cores. >> ----------------------------------------------------------------------------- >> >> Key: SOLR-4300 >> URL: https://issues.apache.org/jira/browse/SOLR-4300 >> Project: Solr >> Issue Type: Bug >> Affects Versions: 4.1, 5.0 >> Reporter: Erick Erickson >> Assignee: Steve Rowe >> Priority: Blocker >> Fix For: 4.1, 5.0 >> >> Attachments: SOLR-4300.patch >> >> >> Yonik pointed out in SOLR-1028 that there is a possible race condition here, >> he's right not to my surprise. Calling it a "blocker" for now so we make a >> decision on it rather than let it fall through the cracks. I should be able >> to get a patch up tonight (Sunday). >> That said, there's potential here to introduce deadlocks, is it worth >> rushing into 4.1? > > -- > This message is automatically generated by JIRA. > If you think it was sent incorrectly, please contact your JIRA administrators > For more information on JIRA, see: http://www.atlassian.com/software/jira > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
