[KUDU-759] Sketchy logic in CatalogManager::BuildLocationsForTablet()

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: M5
• Fix Version/s: 0.10.0
• Component/s: master
• Target Version/s: 1.0.0
• Assignee: Adar Dembo
• Reporter: Mike Percy
• Created: 12/May/15 20:49
• Updated: 14/Jul/16 16:01
• Resolved: 10/Jun/16 01:53

Description

In CatalogManager::BuildLocationsForTablet() there is some funky logic to rebuild the ReplicaState cache when it is not set by reading the ConsensusStatePB quorum members. It's sketchy because it's unclear why that should ever have to happen.

Issue Links

• relates to: KUDU-1353 Master does not rebuild replica information soft state on failover