Adam Storm
Adam Storm
> This would make this RFC unnecessary and I could simply close this PR altogether. Until we have a plan for something that would replace this, we should both leave...
Merging without a second reviewer, as 23.2 backport had the two reviewers.
What qualifies as test-only? We're seeing this on the DRT cluster, which was just upgraded to master ``` W240118 21:26:10.563153 73797719 kv/kvserver/intentresolver/intent_resolver.go:1028 ⋮ [T1,Vsystem,n10,tenant=3,s10,r26124/14:‹/Tenant/3/Table/208/1/115…›] 68980 test-only warning: if you see...
Is this issue something we're still tracking? We see these messages very frequently on the `drt-large` cluster. A random occurrence: ``` W240411 09:38:48.560726 106717155 kv/kvserver/intentresolver/intent_resolver.go:1038 ⋮ [T1,Vsystem,n6,s89,r12983/8:‹/Table/134/1/"@"/12{37/…-55/…}›] 66732 test-only warning:...
Thanks @aadityasondhi. For reference, we hit dozens of these every day on the DRT clusters. I'd suggest that you grep our logs if you're looking for more call sites.