[SOLR-8745] Deprecate ZkStateReader.updateClusterState(), remove uses

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.4.1
• Fix Version/s: 6.1, 7.0
• Component/s: SolrCloud
• Labels: patch performance solrcloud
• Assignee: Shalin Shekhar Mangar
• Reporter: Scott Blum
• Created: 27/Feb/16 00:52
• Updated: 02/Oct/19 17:25
• Resolved: 07/Mar/16 13:14

Description

Forcing a full ZK cluster state update creates a lot of unnecessary work and load at scale. We need to deprecate and remove existing callers.

• The one at the start of ClusterStateUpdater thread is fine, it's a one-time thing.
• The one in OverseerCollectionMessageHandler is getting removed in SOLR-8722
• The rest of them can be replaced with a version that only updates a single collection; not everything!

Patch will be forthcoming.

Issue Links

• breaks: SOLR-9029 regular fails since ZkStateReaderTest.testStateFormatUpdateWithExplicitRefreshLazy