scylladb / cql-stress

10 stars 4 forks source link

Closed muzarski closed 1 year ago

muzarski commented 1 year ago

Motivation

As mentioned in https://github.com/scylladb/cql-stress/issues/54, we should let the user configure the number of connections per shard.

This PR extends both s-b and c-s CLI by option which configures shard connection count.

extended s-b ScyllaBenchArgs by shard_connection_count field. Removed TODO comments and unused variables regarding connections option.
extended c-s NodeOption by shard_connection_count field.

muzarski commented 1 year ago

v2: renamed s-b shard-connection-count CLI argument to connection-count, so the test passes

muzarski commented 1 year ago

v3: fixed review comments