Post review fixes

smartcontractkit · Mar 20, 2024 · 9d9ba13 · 9d9ba13
1 parent 3f42844
commit 9d9ba13
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/core/config/docs/chains-evm.toml b/core/config/docs/chains-evm.toml
@@ -56,6 +56,7 @@ LogPollInterval = '15s' # Default
 LogKeepBlocksDepth = 100000 # Default
 # **ADVANCED**
 # LogPrunePageSize defines size of the page for pruning logs. Controls how many logs/blocks (at most) are deleted in a single prune tick. Default value 0 means no paging, delete everything at once.
+# CCIP uses paging when removing logs to avoid pushing too much presssure on the database so this value should not be set to 0.
 LogPrunePageSize = 10000 # Default
 # **ADVANCED**
 # BackupLogPollerBlockDelay works in conjunction with Feature.LogPoller. Controls the block delay of Backup LogPoller, affecting how far behind the latest finalized block it starts and how often it runs.

diff --git a/docs/CHANGELOG_CCIP.md b/docs/CHANGELOG_CCIP.md
@@ -14,7 +14,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - `OffRamp` offchain config changed:
   - `MaxGasPrice` and `DestMaxGasPrice` will be ignored if specified. DestMaxGasPrice will be read from PriceMax in chain TOML.
 - All LogPoller's filters have now retention defined. This means that the LogPoller will remove logs older than the retention defined per filter. 
-  Additionally, logs are now removed from the database in batches, defined by `LogPrunePageSize` in the toml config
+  Additionally, logs are now removed from the database in batches, defined by `LogPrunePageSize` in the toml config. Default value for all chains is 10_000.
 
 ## 1.4.0 - 2024-02-16
 

diff --git a/docs/CONFIG.md b/docs/CONFIG.md
@@ -6332,6 +6332,7 @@ LogKeepBlocksDepth works in conjunction with Feature.LogPoller. Controls how man
 LogPrunePageSize = 10000 # Default
 ```
 LogPrunePageSize defines size of the page for pruning logs. Controls how many logs/blocks (at most) are deleted in a single prune tick. Default value 0 means no paging, delete everything at once.
+CCIP uses paging when removing logs to avoid pushing too much presssure on the database so this value should not be set to 0.
 
 ### BackupLogPollerBlockDelay
 :warning: **_ADVANCED_**: _Do not change this setting unless you know what you are doing._