Skip to content

fix/recover oom - #1127

Closed
Oppen wants to merge 2 commits into
stagingfrom
fix/recover_oom
Closed

fix/recover oom#1127
Oppen wants to merge 2 commits into
stagingfrom
fix/recover_oom

Conversation

@Oppen

Copy link
Copy Markdown
Contributor

@github-actions

Copy link
Copy Markdown

Changes to gas cost

Generated at commit: 77667a8b9aefd32c0e548e2fe68ac55322831022, compared to commit: 0f535bef99c157e450dba7006d25330afb6f35e7

🧾 Summary (10% most significant diffs)

ContractMethodAvg (+/-)%
AlignedLayerServiceManagercreateNewTask-161 ✅-0.22%

Full diff report 👇
ContractDeployment Cost (+/-)MethodMin (+/-)%Avg (+/-)%Median (+/-)%Max (+/-)%# Calls (+/-)
AlignedLayerServiceManager4,681,797 (+52,382)createNewTask
receive
53,823 (0)
21,169 (0)
0.00%
0.00%
73,728 (-161)
44,690 (-187)
-0.22%
-0.42%
73,987 (0)
45,064 (0)
0.00%
0.00%
74,746 (+12)
45,064 (0)
+0.02%
0.00%
256 (0)
256 (0)
RegistryCoordinatorHarness5,846,016 (+17,263)

@Oppen
Oppen changed the base branch from testnet to stagingSeptember 30, 2024 22:15
@Oppen

Oppen commented Oct 1, 2024

Copy link
Copy Markdown
ContributorAuthor

OOM is one of the few situations recover doesn't attempt to handle 😢

@OppenOppen closed this Oct 1, 2024
@MauroToscano
MauroToscano deleted the fix/recover_oom branch January 2, 2025 18:20
@MauroToscano
MauroToscano restored the fix/recover_oom branch January 2, 2025 18:21
@MauroToscano
MauroToscano deleted the fix/recover_oom branch January 2, 2025 18:22
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@Oppen