Skip to content

Stopping our event parsing logging from OOMing our read-only node#127

Merged
willmeister merged 1 commit intomasterfrom
logging_oom_changes
May 1, 2020
Merged

Stopping our event parsing logging from OOMing our read-only node#127
willmeister merged 1 commit intomasterfrom
logging_oom_changes

Conversation

@willmeister
Copy link

Description

We buffer interim debug log statements while converting EVM event logs to OVM event logs. Sometimes the volume of data logged (buffered) is large enough to cause our node to run out of memory. This makes it so we don't buffer nearly as much info.

Metadata

Fixes

Contributing Agreement

Copy link
Collaborator

@ben-chain ben-chain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@willmeister willmeister merged commit 92ba922 into master May 1, 2020
@willmeister willmeister deleted the logging_oom_changes branch May 1, 2020 20:52
protolambda added a commit to protolambda/optimism that referenced this pull request May 1, 2022
…engine-api

ref impl: Engine API client bindings
bap2pecs pushed a commit to babylonlabs-io/optimism that referenced this pull request Jul 31, 2024
gd-0 pushed a commit to gattaca-com/based-optimism that referenced this pull request May 19, 2025
gd-0 pushed a commit to gattaca-com/based-optimism that referenced this pull request May 19, 2025
theochap pushed a commit that referenced this pull request Dec 10, 2025
* feat(derive): span batch validation

* feat(derive): span batch validity unit tests

* feat(derive): span batch unit tests with acceptance test

* fix(derive): unit tests

* fix(derive): add more unit tests

* feat(derive): span batch validity unit tests for txs

* feat(derive): pipeline builder

* fix(derive): so close :sadge

* fix(derive): ugly refactor

* fix(derive): pipeline construction and trait abstractions

* fix(derive): nit fixes

* fix(derive): temp manual deposit type check
Zena-park added a commit to tokamak-network/optimism that referenced this pull request Dec 30, 2025
theochap pushed a commit that referenced this pull request Jan 15, 2026
<!--
Thank you for your Pull Request. Please provide a description above and
review
the requirements below.

Bug fixes and new features should include tests.

Contributors guide:
https://github.com/alloy-rs/core/blob/main/CONTRIBUTING.md

The contributors guide includes instructions for running rustfmt and
building the
documentation.
-->

<!-- ** Please select "Allow edits from maintainers" in the PR Options
** -->

## Motivation

<!--
Explain the context and why you're making that change. What is the
problem
you're trying to solve? In some cases there is not a problem and this
can be
thought of as being the motivation for your change.
-->

## Solution

<!--
Summarize the solution and provide any necessary context needed to
understand
the code change.
-->

## PR Checklist

- [ ] Added Tests
- [ ] Added Documentation
- [ ] Breaking changes
Sign up for free to 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.

2 participants