set max_connections for tmp db options

EspressoSystems · Nov 20, 2024 · 403826c · 403826c
1 parent 12c8e6b
commit 403826c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/sequencer/src/api/sql.rs b/sequencer/src/api/sql.rs
@@ -468,6 +468,7 @@ mod impl_testable_data_source {
 
             Options {
                 postgres_options: opt,
+                max_connections: 10,
                 ..Default::default()
             }
         }
@@ -480,6 +481,7 @@ mod impl_testable_data_source {
 
             Options {
                 sqlite_options: opt,
+                max_connections: 3,
                 ..Default::default()
             }
         }

diff --git a/sequencer/src/persistence/sql.rs b/sequencer/src/persistence/sql.rs
@@ -183,7 +183,7 @@ pub struct Options {
     #[clap(
         long,
         env = "ESPRESSO_SEQUENCER_DATABASE_MAX_CONNECTIONS",
-        default_value = "25"
+       default_value = "25"
     )]
     pub(crate) max_connections: u32,
 
@@ -273,8 +273,7 @@ impl TryFrom<Options> for Config {
             cfg = cfg.pool(pool);
         }
 
-        // This results in a weird panic so it has been commented out for now.
-        // cfg = cfg.max_connections(opt.max_connections);
+        cfg = cfg.max_connections(opt.max_connections);
         cfg = cfg.idle_connection_timeout(opt.idle_connection_timeout);
         cfg = cfg.min_connections(opt.min_connections);
         cfg = cfg.connection_timeout(opt.connection_timeout);