search

scylladb / scylla-stress-orchestrator

Apache License 2.0
5 stars 9 forks source link

Start Cassandra nodes sequentially waiting for CQL #11

Closed avelanarius closed 3 years ago

avelanarius commented 3 years ago

Previously, Cassandra nodes were started concurrently (all at once). However, this is not a correct approach - the nodes should be started one by one, waiting for each node to fully bootstrap.

This commit changes the behavior to match those best practices. To determine whether a node has fully bootstrapped, checking the CQL port for availability is performed (in both Cassandra and Scylla, CQL port is only started after node bootstrap is fully finished).

Partially addresses #10 (Scylla logic is still wrong - I will address it in next PR dependent on this PR).

avelanarius commented 3 years ago

cc: @pveentjer (for some reason I don't have the rights to assign you as a reviewer)

pveentjer commented 3 years ago

Thanks for the pull requests @avelanarius. Let see if I can fix the rights problem. I'll review the PR now.