Austen
Austen
``` /Local/RangeID/5/r/RangeGCThreshold does not have /Local/Range ``` Could be related to recent changes to speed up descriptor iteration on startup, hitting: https://github.com/cockroachdb/cockroach/blob/3a3c1d8985bc1a744c92089c1666985dfcbc7e3b/pkg/keys/keys.go#L428-L431
The write seems problematic, but still benign. https://github.com/cockroachdb/cockroach/blob/74d5e3e8021e53d27d823ef7f8694d59df8c3414/pkg/kv/kvserver/replica_batch_updates.go#L242-L242 Possibly showing up now due to ba13697aac5a020bfcc6d0ffc10912eacb8b5d2d, which added the batch information cc @lyang24 Details ``` ================== WARNING: DATA RACE Read at...
Duplicate of #125170.
TYFTR! bors r=sumeerbhola
> I'm confused. rangeController is relying on external synchronization (raftMu) for calls to itself (other than the WaitForEval path), which is why it doesn't have a mutex for its replicaMap....
Rebased to resolve conflicts.
Fixed up the last commit to resolve shadowing a var.
Open to suggestions for the migration. I was planning on just hooking these up and having both metrics be enabled at once. We *could* reuse the existing metrics with some...
Should be ready for another look @sumeerbhola.
TYFTR! bors r=sumeerbhola