Skip to content

Comments

Fixing geth submission index to not be 0 for all records#311

Closed
willmeister wants to merge 2 commits intomasterfrom
fixGethSubmissionIndex
Closed

Fixing geth submission index to not be 0 for all records#311
willmeister wants to merge 2 commits intomasterfrom
fixGethSubmissionIndex

Conversation

@willmeister
Copy link

Description

Makes it so all l1_rollup_tx.index_within_submission will not be 0, and for rollup batches with multiple transactions, the correct number will be assigned.

Metadata

Fixes

Contributing Agreement

@tynes tynes closed this Oct 22, 2020
@gakonst gakonst deleted the fixGethSubmissionIndex branch March 18, 2021 15:09
snario pushed a commit that referenced this pull request Apr 14, 2021
…311)

* remove immutable

* add fallback function to deposit eth into l2

* Revert "remove immutable"

This reverts commit 6e8bf7b1871b56085ce96ee73b5acbd4bd04b96c.

* remove conflicting receive function

Co-authored-by: George Hotz <geohot@gmail.com>
Co-authored-by: smartcontracts <kelvinfichter@gmail.com>
protolambda pushed a commit to protolambda/optimism that referenced this pull request May 1, 2022
shenkeyao referenced this pull request in EspressoSystems/optimism-espresso-integration Mar 1, 2025
* test(altda): add test for altda->ethda failover

* feat(batcher): altda->ethda failover when altda is down

* chore: fix typos

* fix(fakeDAServer): handlePut was still handling put when in failover mode

* Fix logs

---------

Co-authored-by: Samuel Laferriere <samlaf92@gmail.com>
QuentinI referenced this pull request in EspressoSystems/optimism-espresso-integration Mar 7, 2025
* test(altda): add test for altda->ethda failover

* feat(batcher): altda->ethda failover when altda is down

* chore: fix typos

* fix(fakeDAServer): handlePut was still handling put when in failover mode

* Fix logs

---------

Co-authored-by: Samuel Laferriere <samlaf92@gmail.com>
QuentinI referenced this pull request in EspressoSystems/optimism-espresso-integration May 27, 2025
* test(altda): add test for altda->ethda failover

* feat(batcher): altda->ethda failover when altda is down

* chore: fix typos

* fix(fakeDAServer): handlePut was still handling put when in failover mode

* Fix logs

---------

Co-authored-by: Samuel Laferriere <samlaf92@gmail.com>
theochap pushed a commit that referenced this pull request Jan 15, 2026
### Description

Adds the missing `op-alloy-registry` to the `op-alloy` aggregator crate
under the `registry` feature flag.
emhane pushed a commit that referenced this pull request Feb 4, 2026
Closes #311 

Performace comparison under load test:

```
previous: 4.873750666s
sorted delete:
3.278444083s 
sorted delete + seperate read (current as per the PR): 2.980065583s
```
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.

Verifier Transaction Batch Ordering

2 participants