chore: move tracing to workspace deps and remove router_env as a dependency of redis_interface (#4717)

This commit is contained in:
Nishant Joshi
2024-05-21 19:48:34 +05:30
committed by GitHub
parent a917776bb8
commit fea2ea6d2c
6 changed files with 11 additions and 9 deletions

View File

@ -23,7 +23,7 @@ use fred::{
},
};
use futures::StreamExt;
use router_env::{instrument, logger, tracing};
use tracing::instrument;
use crate::{
errors,
@ -379,7 +379,7 @@ impl super::RedisConnectionPool {
Some(futures::stream::iter(v))
}
Err(err) => {
logger::error!(?err);
tracing::error!(?err);
None
}
}

View File

@ -26,7 +26,6 @@ use common_utils::errors::CustomResult;
use error_stack::ResultExt;
pub use fred::interfaces::PubsubInterface;
use fred::{interfaces::ClientLike, prelude::EventInterface};
use router_env::logger;
pub use self::types::*;
@ -189,10 +188,10 @@ impl RedisConnectionPool {
let mut error_rx = futures::stream::select_all(error_rxs);
loop {
if let Some(Ok(error)) = error_rx.next().await {
logger::error!(?error, "Redis protocol or connection error");
tracing::error!(?error, "Redis protocol or connection error");
if self.pool.state() == fred::types::ClientState::Disconnected {
if tx.send(()).is_err() {
logger::error!("The redis shutdown signal sender failed to signal");
tracing::error!("The redis shutdown signal sender failed to signal");
}
self.is_redis_available
.store(false, atomic::Ordering::SeqCst);
@ -205,7 +204,7 @@ impl RedisConnectionPool {
pub async fn on_unresponsive(&self) {
let _ = self.pool.clients().iter().map(|client| {
client.on_unresponsive(|server| {
logger::warn!(redis_server =?server.host, "Redis server is unresponsive");
tracing::warn!(redis_server =?server.host, "Redis server is unresponsive");
Ok(())
})
});