Botond Dénes
Botond Dénes
Don't merge yet, v2 is coming.
> Don't merge yet, v2 is coming. Scrap that, this PR is good to go as-is.
> might want to link to [scylladb/scylladb#16669](https://github.com/scylladb/scylladb/issues/16669) with the full path or use `scylladb/scylladb#16669` instead of `#16669` . Fixed.
This has aged out, closing.
The fix would be to set up the limits based on the values of both `iterations` and `durations`: * `iterations` -> unset, `duration` -> unset: run a single iteration; *...
> the "thrift_version" column in system.local table is preserved for backward compatibility, as we could load from an existing system.local table which still contains this clolumn, so we need to...
As a follow-up, we can (maybe) drop `wrapping_interval`, AFAIK its only user is thrift.
> there are still some users of this helper: The read and write path still uses `wrapping_interval` because of thrift. Token metadata indeed uses it and that is not related...
Seen two times yesterday in https://github.com/scylladb/scylladb/pull/18338
> > Seen in unit tests too: #17353 > > Different symptoms. In the unit test we're getting a query failure (driver returns NoHostAvailable). Here we see cassandra-stress hanging. But...