[SOLR-15085] EmbeddedSolrServer calls shutdown on a provided CoreContainer

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 9.0
• Fix Version/s: 9.0, 8.9
• Component/s: Server, SolrJ
• Labels: pull-request-available
• Assignee: Mike Drob
• Reporter: Tim Owen
• Created: 18/Jan/21 12:44
• Updated: 23/Jun/21 13:42
• Resolved: 29/Jan/21 17:17

Description

There are essentially 2 ways to create an EmbeddedSolrServer object, one by passing in a CoreContainer object, and the other way creates one internally on-the-fly. The current behaviour of the close method calls shutdown on the CoreContainer, regardless of where it came from.

I believe this is not good behaviour for a class that doesn't control the lifecycle of the passed-in CoreContainer. In fact, there are 4 cases among the codebase where a subclass of EmbeddedSolrServer is created just to override this behaviour (with a comment saying it's unwanted).

In my use-case I create EmbeddedSolrServer instances for cores as and when I need to work with them, but the CoreContainer exists for the duration. I don't want the whole container shut down when I'm done with just one of its cores. You can workaround it by just not calling close on the EmbeddedSolrServer object, but that's risky especially if you use a try-with-resources as close is called automatically then.

Fix is to keep track of whether the CoreContainer was created internally or not, and only shut it down if internal. I will attach my patch PR.

Issue Links

• links to: GitHub Pull Request #2216