Increase NW batch size .5Mb -> 5Mb (MystenLabs#11073)

tested in private-testnet
futureengco · Apr 22, 2023 · bc1af51 · bc1af51
1 parent 6ff1edf
commit bc1af51
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/crates/sui-config/tests/snapshots/snapshot_tests__network_config_snapshot_matches.snap b/crates/sui-config/tests/snapshots/snapshot_tests__network_config_snapshot_matches.snap
@@ -29,7 +29,7 @@ validator_configs:
         gc_depth: 50
         sync_retry_delay: 5000ms
         sync_retry_nodes: 3
-        batch_size: 500000
+        batch_size: 5000000
         max_batch_delay: 100ms
         block_synchronizer:
           range_synchronize_timeout: 30000ms
@@ -113,7 +113,7 @@ validator_configs:
         gc_depth: 50
         sync_retry_delay: 5000ms
         sync_retry_nodes: 3
-        batch_size: 500000
+        batch_size: 5000000
         max_batch_delay: 100ms
         block_synchronizer:
           range_synchronize_timeout: 30000ms
@@ -197,7 +197,7 @@ validator_configs:
         gc_depth: 50
         sync_retry_delay: 5000ms
         sync_retry_nodes: 3
-        batch_size: 500000
+        batch_size: 5000000
         max_batch_delay: 100ms
         block_synchronizer:
           range_synchronize_timeout: 30000ms
@@ -281,7 +281,7 @@ validator_configs:
         gc_depth: 50
         sync_retry_delay: 5000ms
         sync_retry_nodes: 3
-        batch_size: 500000
+        batch_size: 5000000
         max_batch_delay: 100ms
         block_synchronizer:
           range_synchronize_timeout: 30000ms
@@ -365,7 +365,7 @@ validator_configs:
         gc_depth: 50
         sync_retry_delay: 5000ms
         sync_retry_nodes: 3
-        batch_size: 500000
+        batch_size: 5000000
         max_batch_delay: 100ms
         block_synchronizer:
           range_synchronize_timeout: 30000ms
@@ -449,7 +449,7 @@ validator_configs:
         gc_depth: 50
         sync_retry_delay: 5000ms
         sync_retry_nodes: 3
-        batch_size: 500000
+        batch_size: 5000000
         max_batch_delay: 100ms
         block_synchronizer:
           range_synchronize_timeout: 30000ms
@@ -533,7 +533,7 @@ validator_configs:
         gc_depth: 50
         sync_retry_delay: 5000ms
         sync_retry_nodes: 3
-        batch_size: 500000
+        batch_size: 5000000
         max_batch_delay: 100ms
         block_synchronizer:
           range_synchronize_timeout: 30000ms

diff --git a/narwhal/config/src/lib.rs b/narwhal/config/src/lib.rs
@@ -207,7 +207,7 @@ impl Parameters {
     }
 
     fn default_batch_size() -> usize {
-        500_000
+        5_000_000
     }
 
     fn default_max_batch_delay() -> Duration {

diff --git a/narwhal/config/tests/snapshots/config_tests__parameters.snap b/narwhal/config/tests/snapshots/config_tests__parameters.snap
@@ -10,7 +10,7 @@ expression: parameters
   "gc_depth": 50,
   "sync_retry_delay": "5000ms",
   "sync_retry_nodes": 3,
-  "batch_size": 500000,
+  "batch_size": 5000000,
   "max_batch_delay": "100ms",
   "block_synchronizer": {
     "range_synchronize_timeout": "30000ms",
-Original file line number
+Diff line change
@@ Expand Up / @@ -207,7 +207,7 @@ impl Parameters { @@
         }
         fn default_batch_size() -> usize {
-_000
+_000_000
         }
         fn default_max_batch_delay() -> Duration {
@@ Expand Down @@