Skip to content

Commit 7306165

Browse files
committed
Formatting
1 parent f394d08 commit 7306165

File tree

5 files changed

+38
-19
lines changed

5 files changed

+38
-19
lines changed

lib/src/graph.rs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
use {
33
crate::connection::{ConnectionInfo, Routing},
44
crate::graph::ConnectionPoolManager::Routed,
5-
crate::routing::{RoutedConnectionManager, ClusterRoutingTableProvider},
5+
crate::routing::{ClusterRoutingTableProvider, RoutedConnectionManager},
66
log::debug,
77
};
88

@@ -75,7 +75,10 @@ impl Graph {
7575
)?;
7676
if matches!(info.routing, Routing::Yes(_)) {
7777
debug!("Routing enabled, creating a routed connection manager");
78-
let pool = Routed(RoutedConnectionManager::new(&config, Box::new(ClusterRoutingTableProvider)).await?);
78+
let pool = Routed(
79+
RoutedConnectionManager::new(&config, Box::new(ClusterRoutingTableProvider))
80+
.await?,
81+
);
7982
Ok(Graph {
8083
config: config.into_live_config(),
8184
pool,

lib/src/routing/connection_registry.rs

Lines changed: 17 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
use crate::connection::NeoUrl;
22
use crate::pool::{create_pool, ConnectionPool};
3+
use crate::routing::routing_table_provider::RoutingTableProvider;
34
use crate::routing::Server;
45
use crate::{Config, Error};
56
use dashmap::DashMap;
@@ -8,7 +9,6 @@ use std::sync::Arc;
89
use std::time::Duration;
910
use tokio::sync::mpsc;
1011
use tokio::sync::mpsc::Sender;
11-
use crate::routing::routing_table_provider::RoutingTableProvider;
1212

1313
/// Represents a Bolt server, with its address, port and role.
1414
#[derive(Debug, Clone, PartialEq, Eq, Hash)]
@@ -91,7 +91,7 @@ async fn refresh_routing_table(
9191
uri,
9292
..config.clone()
9393
})
94-
.await?,
94+
.await?,
9595
);
9696
}
9797
registry.connections.retain(|k, _| servers.contains(k));
@@ -180,13 +180,13 @@ impl ConnectionRegistry {
180180

181181
#[cfg(test)]
182182
mod tests {
183-
use std::future::Future;
184-
use std::pin::Pin;
185183
use super::*;
186184
use crate::auth::ConnectionTLSConfig;
187185
use crate::routing::load_balancing::LoadBalancingStrategy;
188186
use crate::routing::RoundRobinStrategy;
189187
use crate::routing::Server;
188+
use std::future::Future;
189+
use std::pin::Pin;
190190

191191
struct TestRoutingTableProvider {
192192
routing_table: RoutingTable,
@@ -199,7 +199,10 @@ mod tests {
199199
}
200200

201201
impl RoutingTableProvider for TestRoutingTableProvider {
202-
fn fetch_routing_table(&self, _: &Config) -> Pin<Box<dyn Future<Output = Result<RoutingTable, Error>> + Send>> {
202+
fn fetch_routing_table(
203+
&self,
204+
_: &Config,
205+
) -> Pin<Box<dyn Future<Output = Result<RoutingTable, Error>> + Send>> {
203206
let routing_table = self.routing_table.clone();
204207
Box::pin(async move { Ok(routing_table) })
205208
}
@@ -251,7 +254,15 @@ mod tests {
251254
tls_config: ConnectionTLSConfig::None,
252255
};
253256
let registry = Arc::new(ConnectionRegistry::default());
254-
let ttl = refresh_routing_table(config.clone(), registry.clone(), Arc::new(Box::new(TestRoutingTableProvider::new(cluster_routing_table)))).await.unwrap();
257+
let ttl = refresh_routing_table(
258+
config.clone(),
259+
registry.clone(),
260+
Arc::new(Box::new(TestRoutingTableProvider::new(
261+
cluster_routing_table,
262+
))),
263+
)
264+
.await
265+
.unwrap();
255266
assert_eq!(ttl, 300);
256267
assert_eq!(registry.connections.len(), 5);
257268
let strategy = RoundRobinStrategy::default();

lib/src/routing/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,4 +165,4 @@ use crate::routing::connection_registry::BoltServer;
165165
use crate::{Database, Version};
166166
pub use load_balancing::round_robin_strategy::RoundRobinStrategy;
167167
pub use routed_connection_manager::RoutedConnectionManager;
168-
pub use routing_table_provider::ClusterRoutingTableProvider;
168+
pub use routing_table_provider::ClusterRoutingTableProvider;

lib/src/routing/routed_connection_manager.rs

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
use crate::pool::ManagedConnection;
2-
use crate::routing::connection_registry::{start_background_updater, BoltServer, ConnectionRegistry, RegistryCommand};
2+
use crate::routing::connection_registry::{
3+
start_background_updater, BoltServer, ConnectionRegistry, RegistryCommand,
4+
};
35
use crate::routing::load_balancing::LoadBalancingStrategy;
6+
use crate::routing::routing_table_provider::RoutingTableProvider;
47
use crate::routing::RoundRobinStrategy;
58
#[cfg(feature = "unstable-bolt-protocol-impl-v2")]
69
use crate::{Config, Error, Operation};
@@ -10,7 +13,6 @@ use log::{debug, error};
1013
use std::sync::Arc;
1114
use std::time::Duration;
1215
use tokio::sync::mpsc::Sender;
13-
use crate::routing::routing_table_provider::RoutingTableProvider;
1416

1517
#[derive(Clone)]
1618
pub struct RoutedConnectionManager {
@@ -23,7 +25,10 @@ pub struct RoutedConnectionManager {
2325
}
2426

2527
impl RoutedConnectionManager {
26-
pub async fn new(config: &Config, provider: Box<dyn RoutingTableProvider>) -> Result<Self, Error> {
28+
pub async fn new(
29+
config: &Config,
30+
provider: Box<dyn RoutingTableProvider>,
31+
) -> Result<Self, Error> {
2732
let backoff = Arc::new(
2833
ExponentialBackoffBuilder::new()
2934
.with_initial_interval(Duration::from_millis(1))
@@ -34,7 +39,8 @@ impl RoutedConnectionManager {
3439
);
3540

3641
let connection_registry = Arc::new(ConnectionRegistry::default());
37-
let channel = start_background_updater(config, connection_registry.clone(), provider.into()).await;
42+
let channel =
43+
start_background_updater(config, connection_registry.clone(), provider.into()).await;
3844
Ok(RoutedConnectionManager {
3945
load_balancing_strategy: Arc::new(RoundRobinStrategy::default()),
4046
bookmarks: Arc::new(Mutex::new(vec![])),

lib/src/routing/routing_table_provider.rs

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
use std::future::Future;
2-
use std::pin::Pin;
3-
use crate::{Config, Error};
41
use crate::connection::{Connection, ConnectionInfo};
52
use crate::routing::{RouteBuilder, RoutingTable};
3+
use crate::{Config, Error};
4+
use std::future::Future;
5+
use std::pin::Pin;
66

77
pub(crate) trait RoutingTableProvider: Send + Sync {
88
fn fetch_routing_table(
@@ -31,8 +31,7 @@ impl RoutingTableProvider for ClusterRoutingTableProvider {
3131
if let Some(db) = config.db.clone() {
3232
builder = builder.with_db(db);
3333
}
34-
connection
35-
.route(builder.build(connection.version())).await
34+
connection.route(builder.build(connection.version())).await
3635
})
3736
}
38-
}
37+
}

0 commit comments

Comments
 (0)