[SOLR-15959] Deprecate loading of solr.xml from ZK

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 9.2
• Assignee: Jan Høydahl
• Reporter: Jan Høydahl
• Created: 27/Jan/22 00:28
• Updated: 13/Sep/23 12:56
• Resolved: 22/Feb/23 10:04

Description

In SOLR-7735, five years ago (committed by me) we allowed loading of solr.xml from zookeeper.

Part of the motivation back then was to be able to put solr.xml in a central place and not being forced to initialize SOLR_HOME with that file, similar to the goal of SOLR-9575. And perhaps also an easy way to update solr.xml.

However, I think it may have been a mistake that should be deprecated and ultimately removed. Why? The solr.xml file is the "node config" for solr. Where you set node-level props that need to be there at early node startup. So even if you change the file in ZK, nodes will have not way of reloading those plugins dynamically without a restart

The file may even contain local-node-specific settings that shuold differ from node to node. Especially during rolling upgrade, it may be important to be able to have a different solr.xml on the new nodes and on the old ones. With property substitution you can do some of that, but you cannot add XML tags with a sysprop.

Issue Links

• is depended upon by: SOLR-16975 Remove deprecated feature to load solr.xml from ZK (main only)
• relates to: SOLR-4718 Allow solr.xml to be stored in zookeeper
• relates to: SOLR-7735 Look for solr.xml in Zookeeper by default
• links to: GitHub Pull Request #1362