round_robin.rs 4.22 KB
Newer Older
1
2
3
4
//! Round-robin load balancing policy

use super::{get_healthy_worker_indices, LoadBalancingPolicy};
use crate::core::Worker;
5
use crate::metrics::RouterMetrics;
6
use std::sync::atomic::{AtomicUsize, Ordering};
7
use std::sync::Arc;
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27

/// Round-robin selection policy
///
/// Selects workers in sequential order, cycling through all healthy workers.
#[derive(Debug, Default)]
pub struct RoundRobinPolicy {
    counter: AtomicUsize,
}

impl RoundRobinPolicy {
    pub fn new() -> Self {
        Self {
            counter: AtomicUsize::new(0),
        }
    }
}

impl LoadBalancingPolicy for RoundRobinPolicy {
    fn select_worker(
        &self,
28
        workers: &[Arc<dyn Worker>],
29
30
31
32
33
34
35
36
37
38
39
        _request_text: Option<&str>,
    ) -> Option<usize> {
        let healthy_indices = get_healthy_worker_indices(workers);

        if healthy_indices.is_empty() {
            return None;
        }

        // Get and increment counter atomically
        let count = self.counter.fetch_add(1, Ordering::Relaxed);
        let selected_idx = count % healthy_indices.len();
40
        let worker = workers[healthy_indices[selected_idx]].url();
41

42
43
        RouterMetrics::record_processed_request(worker);
        RouterMetrics::record_policy_decision(self.name(), worker);
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
        Some(healthy_indices[selected_idx])
    }

    fn name(&self) -> &'static str {
        "round_robin"
    }

    fn reset(&self) {
        self.counter.store(0, Ordering::Relaxed);
    }

    fn as_any(&self) -> &dyn std::any::Any {
        self
    }
}

#[cfg(test)]
mod tests {
    use super::*;
    use crate::core::{BasicWorker, WorkerType};

    #[test]
    fn test_round_robin_selection() {
        let policy = RoundRobinPolicy::new();
68
69
        let workers: Vec<Arc<dyn Worker>> = vec![
            Arc::new(BasicWorker::new(
70
71
72
                "http://w1:8000".to_string(),
                WorkerType::Regular,
            )),
73
            Arc::new(BasicWorker::new(
74
75
76
                "http://w2:8000".to_string(),
                WorkerType::Regular,
            )),
77
            Arc::new(BasicWorker::new(
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
                "http://w3:8000".to_string(),
                WorkerType::Regular,
            )),
        ];

        // Should select workers in order: 0, 1, 2, 0, 1, 2, ...
        assert_eq!(policy.select_worker(&workers, None), Some(0));
        assert_eq!(policy.select_worker(&workers, None), Some(1));
        assert_eq!(policy.select_worker(&workers, None), Some(2));
        assert_eq!(policy.select_worker(&workers, None), Some(0));
        assert_eq!(policy.select_worker(&workers, None), Some(1));
    }

    #[test]
    fn test_round_robin_with_unhealthy_workers() {
        let policy = RoundRobinPolicy::new();
94
95
        let workers: Vec<Arc<dyn Worker>> = vec![
            Arc::new(BasicWorker::new(
96
97
98
                "http://w1:8000".to_string(),
                WorkerType::Regular,
            )),
99
            Arc::new(BasicWorker::new(
100
101
102
                "http://w2:8000".to_string(),
                WorkerType::Regular,
            )),
103
            Arc::new(BasicWorker::new(
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
                "http://w3:8000".to_string(),
                WorkerType::Regular,
            )),
        ];

        // Mark middle worker as unhealthy
        workers[1].set_healthy(false);

        // Should skip unhealthy worker: 0, 2, 0, 2, ...
        assert_eq!(policy.select_worker(&workers, None), Some(0));
        assert_eq!(policy.select_worker(&workers, None), Some(2));
        assert_eq!(policy.select_worker(&workers, None), Some(0));
        assert_eq!(policy.select_worker(&workers, None), Some(2));
    }

    #[test]
    fn test_round_robin_reset() {
        let policy = RoundRobinPolicy::new();
122
123
        let workers: Vec<Arc<dyn Worker>> = vec![
            Arc::new(BasicWorker::new(
124
125
126
                "http://w1:8000".to_string(),
                WorkerType::Regular,
            )),
127
            Arc::new(BasicWorker::new(
128
129
130
131
132
133
134
135
136
137
138
139
140
141
                "http://w2:8000".to_string(),
                WorkerType::Regular,
            )),
        ];

        // Advance the counter
        assert_eq!(policy.select_worker(&workers, None), Some(0));
        assert_eq!(policy.select_worker(&workers, None), Some(1));

        // Reset should start from beginning
        policy.reset();
        assert_eq!(policy.select_worker(&workers, None), Some(0));
    }
}