fix(blockifier, cairo_native): fix stack size default config value (#…

…3880)
starkware-libs · Feb 3, 2025 · 80bc990 · 80bc990
1 parent 2165f34
commit 80bc990
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/config/sequencer/default_config.json b/config/sequencer/default_config.json
@@ -112,7 +112,7 @@
   "batcher_config.block_builder_config.execute_config.stack_size": {
     "description": "The thread stack size (proportional to the maximal gas of a transaction).",
     "privacy": "Public",
-    "value": 0
+    "value": 62914560
   },
   "batcher_config.block_builder_config.tx_chunk_size": {
     "description": "The size of the transaction chunk.",

diff --git a/crates/blockifier/src/blockifier/config.rs b/crates/blockifier/src/blockifier/config.rs
@@ -6,12 +6,11 @@ use serde::{Deserialize, Serialize};
 #[cfg(feature = "cairo_native")]
 use starknet_sierra_multicompile::config::SierraCompilationConfig;
 
-#[cfg(any(test, feature = "testing", feature = "native_blockifier"))]
 use crate::blockifier::transaction_executor::DEFAULT_STACK_SIZE;
 use crate::state::contract_class_manager::DEFAULT_COMPILATION_REQUEST_CHANNEL_SIZE;
 use crate::state::global_cache::GLOBAL_CONTRACT_CACHE_SIZE_FOR_TEST;
 
-#[derive(Clone, Debug, Default, Deserialize, PartialEq, Serialize)]
+#[derive(Clone, Debug, Deserialize, PartialEq, Serialize)]
 pub struct TransactionExecutorConfig {
     pub concurrency_config: ConcurrencyConfig,
     pub stack_size: usize,
@@ -26,6 +25,12 @@ impl TransactionExecutorConfig {
     }
 }
 
+impl Default for TransactionExecutorConfig {
+    fn default() -> Self {
+        Self { concurrency_config: ConcurrencyConfig::default(), stack_size: DEFAULT_STACK_SIZE }
+    }
+}
+
 impl SerializeConfig for TransactionExecutorConfig {
     fn dump(&self) -> BTreeMap<ParamPath, SerializedParam> {
         let mut dump = append_sub_config_name(self.concurrency_config.dump(), "concurrency_config");