basho / riak_kv

Riak Key/Value Store
Apache License 2.0
653 stars 233 forks source link

Dynamic changing of snk workers leads to unpredictable behaviour #1867

Open martinsumner opened 1 year ago

martinsumner commented 1 year ago

The riak_kv_replrtq_snk worker_count is intended to help control the pace at which replication references are fetched by a sink cluster. However, reducing the sink worker count can lead to a situation where fetching is no longer limited by the worker count, but instead by the per_peer_limit * count(peers).

This is a problem caused by the deferred_queue_length not being reset not being reset when the worker_count is updated.