Marcus Gartner
Marcus Gartner
I think that could make sense for certain types. The only one that comes immediately to mind is `Litmus.Type.List`. E.g. if you wanted to validate a list of strings that...
> Was this section always incorrect (should this PR technically apply to all CockroachDB versions), or did it become incorrect at a specific version? Yes, I believe this has always...
Yes, that's an assumption we make by default for `REGIONAL BY ROW` tables that have unique keys with a default value of `gen_random_uuid()` or are updated/inserted with `gen_random_uuid()`. There's an...
@kvoli Can the distsender change that is causing this be reverted, or is there some setting we can temporarily disable to avoid these failures? This is failing logic tests several...
One interesting data point is that we tend to see a cluster of these failures in separate nightlies at about the same. Sometimes they occur at the same time on...
@rafiss might have more of an opinion than I do about what would be most appropriate for these tests. My main concern is that this could be a real issue...
Friendly ping @DrewKimball
This reproduces quickly with `./dev test pkg/sql/logictest/tests/local -f TestLogic_select_for_update --stress`. It looks like something in the last ~50 commits on master is causing this. I'm bisecting now.
I'm having trouble bisecting because I think I was overly optimistic on how quickly the failure occurs under stress. Sometimes it will fail in the first few seconds, but other...
ALso occurred in #132962 and #132990.