Make sure all values pulled from solrcore.properties have defaults, make solrconfig_extra,xml optional [#1984442]

Especially people updating the codule and copying in the new conf files may omit the solrcore.properties file.

We should make sure all variables in solrconfig.xml have defaults, so that it works correctly without that file.

I'm also seeing a failure when the solrconfig_extra.xml file is omitted. That must be optional!

Comment	File	Size	Author
#1	1984442-1.patch	16.01 KB	pwolanin

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

pwolanin CreditAttribution: pwolanin commented 1 May 2013 at 20:52

Status:

Active

» Needs review

File	Size
1984442-1.patch	16.01 KB

Comment #2

Nick_vh

he/him

Ghent

CreditAttribution: Nick_vh commented 1 May 2013 at 20:54

While we are working on this, we should add buildonOptimize back for backwards compatibilty. We can do it in another issue but it seems reasonable to make the small change here

edit: We decided to do that in another issue

Comment #3

pwolanin CreditAttribution: pwolanin commented 1 May 2013 at 21:29

Title:	Make sure all values pulled from solrcore.properties have defaults	» Make sure all values pulled from solrcore.properties have defaults, make solrconfig_extra,xml optional
Status:	Needs review	» Fixed

committed