Skip to content

Handle Consolidation Processing Edge Case#15122

Merged
nisdas merged 3 commits into
developfrom
cleanUpConsolidationProcessing
Apr 3, 2025
Merged

Handle Consolidation Processing Edge Case#15122
nisdas merged 3 commits into
developfrom
cleanUpConsolidationProcessing

Conversation

@nisdas

@nisdas nisdas commented Apr 3, 2025

Copy link
Copy Markdown
Contributor

What type of PR is this?

Bug Fix

What does this PR do? Why is it needed?

  • In the event we have the maximum amount of consolidations included in our pending queue in the state, we will exit the consolidation processing loop. This PR makes sure that we continue processing new consolidation requests to handle new compounding consolidations.
  • Regression test

Which issues(s) does this PR fix?

N.A

Other notes for review

Acknowledgements

@nisdas nisdas added the Electra label Apr 3, 2025
@nisdas nisdas requested a review from a team as a code owner April 3, 2025 15:17
@nisdas nisdas requested review from kasey, rkapka and terencechain April 3, 2025 15:17
@nisdas nisdas added this pull request to the merge queue Apr 3, 2025
Merged via the queue into develop with commit 0c2464c Apr 3, 2025
@nisdas nisdas deleted the cleanUpConsolidationProcessing branch April 3, 2025 16:17
fernantho pushed a commit to fernantho/prysm that referenced this pull request Sep 26, 2025
* Clean it up

* Add regression test case

* changelog
fernantho pushed a commit to fernantho/prysm that referenced this pull request Sep 26, 2025
* Clean it up

* Add regression test case

* changelog
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.

3 participants