fix: Add stacker to manage stack growth in the parser

Just in case another environment has a smaller stack size than my local environment we also grow the stack automatically. We still have the depth limit to prevent the stack from growing unbounded
influxdata · Sep 9, 2022 · 6a60a15 · 6a60a15
1 parent da1b031
commit 6a60a15
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 1 deletion.
diff --git a/libflux/Cargo.lock b/libflux/Cargo.lock
diff --git a/libflux/flux-core/Cargo.toml b/libflux/flux-core/Cargo.toml
@@ -60,6 +60,7 @@ regex = "1.5.5"
 serde = { version = "^1.0.136", features = ["derive", "rc"] }
 serde_derive = "^1.0.136"
 serde_json = "1.0.79"
+stacker = "0.1"
 structopt = "0.3.26"
 thiserror = "1"
 tempfile = { version = "3.3.0", optional = true }

diff --git a/libflux/flux-core/src/parser/mod.rs b/libflux/flux-core/src/parser/mod.rs
@@ -911,7 +911,8 @@ impl<'input> Parser<'input> {
             self.errs.push(format!("Program is nested too deep"));
             None
         } else {
-            Some(f(self))
+            // Numbers are arbitrary and were just picked from the stack docs
+            Some(stacker::maybe_grow(32 * 1024, 1024 * 1024, || f(self)))
         };
 
         self.depth -= 1;