Skip to content

Deprecate `LoadBalancer#select_caught_up_hosts`

What does this MR do?

Implements #335107 (closed).

LoadBalancer#select_caught_up_hosts has unnecessary logic managing the host list which we don't use (except selecting a single sample host from it). Thus, it could be replaced with LoadBalancer#select_up_to_date_host.

Because LoadBalancer#select_caught_up_hosts was implemented as a response to a particular issue with the merge requests - MR - I consider that it would be safer to deprecate this logic via the Feature Flag.

FF rollout issue: #335109 (closed)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

N/A

Related to #335107 (closed)

Edited by Aleksei Lipniagov

Merge request reports