-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
kvserver: setup ranges in large unsplittable replicate test #112990
kvserver: setup ranges in large unsplittable replicate test #112990
Conversation
Essential CI failure unrelated, I'll rebase later. |
e6bd351
to
6fab2c5
Compare
@@ -1774,8 +1774,6 @@ func TestLargeUnsplittableRangeReplicate(t *testing.T) { | |||
base.TestClusterArgs{ | |||
ReplicationMode: base.ReplicationAuto, | |||
ServerArgs: base.TestServerArgs{ | |||
ScanMinIdleTime: time.Millisecond, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was redundant, the test already force scan & processes every range within the SucceedsSoon
loops below.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In addition to enqueuing on span config updates.
This reverts commit 9d4a50c.
`TestLargeUnsplittableRangeReplicate` would fail when the SQL query to show the table's replicas returned no results for the unsplittable range being tested. No results would be returned when the table split was delayed. Update the test to insert a split at the first row, as well as the existing split at the second row. This avoids the first range starting with 5 replicas and the test's reliance on timely span config splits. Resolves: cockroachdb#112774 Release note: None
6fab2c5
to
ad54149
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TYFTR bors r=andrewbaptist |
Build succeeded: |
TestLargeUnsplittableRangeReplicate
would fail when the SQL query toshow the table's replicas returned no results for the unsplittable range
being tested. No results would be returned when the table split was
delayed.
Update the test to insert a split at the first row, as well as the
existing split at the second row. This avoids the first range starting
with 5 replicas and the test's reliance on timely span config splits.
The previous deflake attempt is reverted.
Resolves: #112774
Release note: None