i'm always a fan of quick fixes ... even if they are only temporary ...
but like i said, i'm confused as to why you wanted to avoid #1 or why #2
would breakthings.
sorry, my object to #1 was a synchronized map lookup. a simple
synchronized call should not hurt.
#2 breaks the Jetty test -- I'm looking for a way to rewrite it that
does not, but nothing is obvious yet. The basic issue is that the
testHarness creates a new SolrCore for each test. Jetty runner is
initialized with:
root.addFilter( SolrDispatchFilter.class, "*", Handler.REQUEST );
then jetty instantiates the DispatchFilter which calls getSolrCore() and
needs to get the *last* one initialized.
- - - - -
I'll commit option #1, and keep an eye on this for SOLR-350
ryan
