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

Improve cassandra-stress to support multi DC testing #16

Open
tzach opened this issue Feb 21, 2016 · 6 comments
Open

Improve cassandra-stress to support multi DC testing #16

tzach opened this issue Feb 21, 2016 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@tzach
Copy link

tzach commented Feb 21, 2016

cassandra-stress (c-s) is our go to tool for benchmark.
It will be useful if we can use it to benchmark Scylla/Cassandra cluster on failures, in particular DC failures.

c-s chose which DC to work in the client builder code.
Currently c-s is using the default, DCAwareRoundRobinPolicy() which creates a new datacenter aware round robin policy, with one, random DC.

There is also a DCAwareRoundRobinPolicy(String localDc, int usedHostsPerRemoteDc)[1][2] which provide the client fail over to other DC, c-s does not use it[2]

I suggest a new c-s options -remote-dc n to set the number of remote DC hosts to connect to.

[1] http://docs.datastax.com/en/drivers/java/2.2/com/datastax/driver/core/policies/DCAwareRoundRobinPolicy.html
[2] https://github.com/apache/cassandra/blob/b51f9495e5f1dc9573aa08139009569a1916f82f/tools/stress/src/org/apache/cassandra/stress/util/JavaDriverClient.java#L113
[3] https://github.com/datastax/java-driver/blob/2.1/driver-core/src/main/java/com/datastax/driver/core/policies/DCAwareRoundRobinPolicy.java

@tzach tzach added the enhancement New feature or request label Feb 21, 2016
@penberg
Copy link

penberg commented Mar 3, 2016

@tzach I sent a patch on the mailing list. Help needed with testing it.

@tzach tzach assigned tzach and unassigned penberg May 2, 2016
@asias
Copy link

asias commented Oct 31, 2016

@penberg What is the status of the patch. Is the patch merged in https://github.com/scylladb/scylla-tools-java? How can we use it?

@penberg
Copy link

penberg commented Nov 7, 2016

@asias It's not merged because nobody was able to test it.

@amoskong
Copy link

@penberg what's the patch? (mail subject)
Do you have build packages / AMI / etc? If not I can build by myself.
I added support for multi-dc in SCT (AWS & GCE), so we can test your patch.

@penberg
Copy link

penberg commented Jun 10, 2017

The patch subject is "[PATCH scylla-tools v1] cassandra-stress: Add new '-remote-dc' command line option".

@amoskong
Copy link

amoskong commented Sep 6, 2017

Asked pekka to rebase the patch to latest code, I would like to test it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants