-
Notifications
You must be signed in to change notification settings - Fork 3.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
roachtest: remove version persistence hack after v2.0.5 is out #27717
Labels
A-kv-client
Relating to the KV client and the KV interface.
C-cleanup
Tech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.
Milestone
Comments
This was referenced Jul 18, 2018
tbg
added
the
C-cleanup
Tech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.
label
Jul 22, 2018
tbg
added a commit
to tbg/cockroach
that referenced
this issue
Sep 20, 2018
Closes cockroachdb#27717. Release note: None
craig bot
pushed a commit
that referenced
this issue
Sep 20, 2018
30405: roachtest: mark acceptance as stable r=petermattis a=tschottdorf all of its subtests are already stable, but in running a test locally I noticed that the top-level test was marked as passing as unstable. I'm not sure, but this might mean that the top-level test would actually not fail? Either way, better to mark it as stable explicitly. We should also spend some thought on how diverging notions of Stable in sub vs top level test are treated, not sure that this is well-defined. Release note: None 30446: opt: fix panic when srf used with GROUP BY r=rytaft a=rytaft Instead of panicking, we now throw an appropriate error. Fixes #30412 Release note (bug fix): Fixed a panic that occurred when a generator function such as unnest was used in the SELECT list in the presence of GROUP BY. 30450: roachtest: remove now-unnecessary hack r=petermattis a=tschottdorf Closes #27717. Release note: None 30451: storage: give TestReplicateRemovedNodeDisruptiveElection more time r=petermattis a=tschottdorf Perhaps: Fixes #27253. Release note: None 30452: storage: de-flake TestReplicaIDChangePending r=petermattis a=tschottdorf setReplicaID refreshes the proposal and was thus synchronously writing to the commandProposed chan. This channel could have filled up due to an earlier reproposal already, deadlocking the test. Fixes #28132. Release note: None 30455: testcluster: improve AddReplicas check r=petermattis a=tschottdorf AddReplicas was verifying that a replica had indeed been added, but there's no guarantee that the replicate queue wouldn't have removed it in the meantime. Attempt to work around this somewhat. The real solution is not to provide that guarantee, but some tests likely rely on it (and the failure is extremely rare, i.e. the new for loop basically never runs). Observed in #28368. Release note: None 30456: storage: unskip TestClosedTimestampCanServe for non-race r=petermattis a=tschottdorf Fixes #28607. Release note: None Co-authored-by: Tobias Schottdorf <tobias.schottdorf@gmail.com> Co-authored-by: Rebecca Taft <becca@cockroachlabs.com>
tbg
added a commit
to tbg/cockroach
that referenced
this issue
Sep 21, 2018
Closes cockroachdb#27717. Release note: None
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
A-kv-client
Relating to the KV client and the KV interface.
C-cleanup
Tech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.
This is a reminder to myself. See #27639 for context.
The text was updated successfully, but these errors were encountered: