Ensure OLAP config has enough workers for pg_cron (#867)

Co-authored-by: Adam Hendel <[email protected]>
tembo-io · Jul 10, 2024 · 605012f · 605012f
1 parent 0606901
commit 605012f
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/tembo-stacks/Cargo.lock b/tembo-stacks/Cargo.lock
diff --git a/tembo-stacks/Cargo.toml b/tembo-stacks/Cargo.toml
@@ -1,7 +1,7 @@
 [package]
 name = "tembo-stacks"
 description = "Tembo Stacks for Postgres"
-version = "0.11.0"
+version = "0.11.1"
 authors = ["tembo.io"]
 edition = "2021"
 license = "Apache-2.0"

diff --git a/tembo-stacks/src/stacks/config_engines.rs b/tembo-stacks/src/stacks/config_engines.rs
@@ -168,7 +168,7 @@ fn olap_max_parallel_workers(cpu: f32) -> i32 {
 }
 
 fn olap_max_worker_processes(cpu: f32) -> i32 {
-    i32::max(1, cpu.round() as i32)
+    i32::max(1, cpu.round() as i32) + 1 // add one for cron
 }
 
 // olap formula for maintenance_work_mem
@@ -535,7 +535,7 @@ mod tests {
         assert_eq!(configs[5].name, "max_wal_size");
         assert_eq!(configs[5].value.to_string(), "2GB");
         assert_eq!(configs[6].name, "max_worker_processes");
-        assert_eq!(configs[6].value.to_string(), "4");
+        assert_eq!(configs[6].value.to_string(), "5");
         assert_eq!(configs[7].name, "shared_buffers");
         assert_eq!(configs[7].value.to_string(), "4096MB");
         assert_eq!(configs[8].name, "work_mem");
@@ -568,7 +568,7 @@ mod tests {
         assert_eq!(configs[5].name, "max_wal_size");
         assert_eq!(configs[5].value.to_string(), "2GB");
         assert_eq!(configs[6].name, "max_worker_processes");
-        assert_eq!(configs[6].value.to_string(), "1");
+        assert_eq!(configs[6].value.to_string(), "2");
         assert_eq!(configs[7].name, "shared_buffers");
         assert_eq!(configs[7].value.to_string(), "2048MB");
         assert_eq!(configs[8].name, "work_mem");