From 34b52467f07979e240f31080efb4edf4e5fccf27 Mon Sep 17 00:00:00 2001 From: Anna Khanova Date: Tue, 4 Jun 2024 20:45:30 +0200 Subject: [PATCH] Add comments --- proxy/src/console/provider.rs | 2 +- proxy/src/rate_limiter/limit_algorithm.rs | 6 ++++++ proxy/src/rate_limiter/limit_algorithm/aimd.rs | 2 ++ 3 files changed, 9 insertions(+), 1 deletion(-) diff --git a/proxy/src/console/provider.rs b/proxy/src/console/provider.rs index 634ec9042c..cab15cb92c 100644 --- a/proxy/src/console/provider.rs +++ b/proxy/src/console/provider.rs @@ -509,7 +509,7 @@ impl ApiLocks { self.metrics .semaphore_acquire_seconds .observe(now.elapsed().as_secs_f64()); - + info!("acquired permit {:?}", now.elapsed().as_secs_f64()); Ok(WakeComputePermit { permit: permit? }) } diff --git a/proxy/src/rate_limiter/limit_algorithm.rs b/proxy/src/rate_limiter/limit_algorithm.rs index 3842ce269e..b5604b32e6 100644 --- a/proxy/src/rate_limiter/limit_algorithm.rs +++ b/proxy/src/rate_limiter/limit_algorithm.rs @@ -92,6 +92,12 @@ impl LimiterInner { } fn take(&mut self, ready: &Notify) -> Option<()> { + tracing::info!( + "available: {}, in_flight: {}, limit: {}", + self.available, + self.in_flight, + self.limit + ); if self.available >= 1 { self.available -= 1; self.in_flight += 1; diff --git a/proxy/src/rate_limiter/limit_algorithm/aimd.rs b/proxy/src/rate_limiter/limit_algorithm/aimd.rs index ccc9c42420..812f147a4f 100644 --- a/proxy/src/rate_limiter/limit_algorithm/aimd.rs +++ b/proxy/src/rate_limiter/limit_algorithm/aimd.rs @@ -28,6 +28,8 @@ pub struct Aimd { impl LimitAlgorithm for Aimd { fn update(&self, old_limit: usize, sample: Sample) -> usize { use Outcome::*; + tracing::info!(old_limit, "updating limit"); + tracing::info!(sample.in_flight, "in flight"); match sample.outcome { Success => { let utilisation = sample.in_flight as f32 / old_limit as f32;