Skip to content
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

storage: enable pre-vote by default #17808

Merged
merged 1 commit into from
Aug 23, 2017

Conversation

irfansharif
Copy link
Contributor

@irfansharif irfansharif commented Aug 22, 2017

Tested on blue under heavy chaos for half a week, tested on ultramarine for a week. Part of #16950.

crontab for blue:

0  0-23/8 * * * /home/cockroach/chaos_runner.sh 60 1200
48 0-23/8 * * * /home/cockroach/chaos_runner.sh 60 1200
36 1-23/8 * * * /home/cockroach/chaos_runner.sh 60 1200
24 2-23/8 * * * /home/cockroach/chaos_runner.sh 60 1200
12 3-23/8 * * * /home/cockroach/chaos_runner.sh 60 1200
0  4-23/8 * * * /home/cockroach/chaos_runner.sh 60 1200
48 4-23/8 * * * /home/cockroach/chaos_runner.sh 60 1200
36 5-23/8 * * * /home/cockroach/chaos_runner.sh 60 1200
24 6-23/8 * * * /home/cockroach/chaos_runner.sh 60 1200
12 7-23/8 * * * /home/cockroach/chaos_runner.sh 60 1200

crontab for ultramarine:

0  * * * * /home/cockroach/chaos_runner.sh 60 600
14 * * * * /home/cockroach/chaos_runner.sh 60 600
29 * * * * /home/cockroach/chaos_runner.sh 60 600
44 * * * * /home/cockroach/chaos_runner.sh 60 600

image

Last ultramarine chaos run can be found here. Nothing jumps out to me. Ditto for the logs on ultramarine, still present (blue was wiped out for 1.0.5 testing).

+cc @bdarnell

@irfansharif irfansharif requested a review from a team August 22, 2017 15:31
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@bdarnell
Copy link
Contributor

There are big swings in raft append and heartbeat traffic every time a node goes down.

image

We don't see swings that severe on blue:

image

Is this because of differing configs (loadgens, chaos, etc) between blue and ultramarine? Can you run ultramarine for a while with this change reverted so we can have apples-to-apples comparisons?

@irfansharif
Copy link
Contributor Author

Here's a run from last night: link

The first half of the graphs (between 20:28:30 - 3:43:30) is running with the pre-vote changes enabled, the second half is without.

image
image

The larger variations noted above was owed to differences in workload.

0  0-23/2 * * * /home/cockroach/chaos_runner.sh 60 600
29 0-23/2 * * * /home/cockroach/chaos_runner.sh 60 600
0 1-23/2 * * * /home/cockroach/chaos_runner.sh 60 600
29 1-23/2 * * * /home/cockroach/chaos_runner.sh 60 600

This run is running kv --concurrency 128 --tolerate-errors --min-block-bytes=8 --max-block-bytes=128 --read-percent=25 on each node.

@irfansharif
Copy link
Contributor Author

🤞

@irfansharif irfansharif merged commit a9b9e1d into cockroachdb:master Aug 23, 2017
@irfansharif irfansharif deleted the enable-pre-vote branch August 25, 2017 02:46
@irfansharif irfansharif restored the enable-pre-vote branch September 18, 2017 16:47
@irfansharif irfansharif deleted the enable-pre-vote branch November 18, 2019 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants