Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prune programs deployed in duplicate unconfirmed slot #32999

Merged
merged 2 commits into from
Aug 25, 2023

Conversation

pgarg66
Copy link
Contributor

@pgarg66 pgarg66 commented Aug 25, 2023

Problem

Programs could be deployed in a duplicate slot. Once the slot is determined to be unconfirmed, the cache should be pruned of these programs.

Summary of Changes

Added a new method to prune cache for the programs deployed in a given slot
Call the method from replay stage from duplicate unconfirmed slot handler.

Fixes #

@pgarg66 pgarg66 marked this pull request as ready for review August 25, 2023 17:51
@pgarg66 pgarg66 added the v1.16 PRs that should be backported to v1.16 label Aug 25, 2023
@pgarg66 pgarg66 merged commit dbe4017 into solana-labs:master Aug 25, 2023
16 checks passed
@pgarg66 pgarg66 deleted the prune-programs-dup-slot branch August 25, 2023 18:02
mergify bot pushed a commit that referenced this pull request Aug 25, 2023
* Prune programs deployed in duplicate unconfirmed slot

* unit test

(cherry picked from commit dbe4017)

# Conflicts:
#	core/src/replay_stage.rs
pgarg66 added a commit that referenced this pull request Aug 25, 2023
* Prune programs deployed in duplicate unconfirmed slot

* unit test

(cherry picked from commit dbe4017)
mergify bot added a commit that referenced this pull request Aug 25, 2023
…t of #32999) (#33003)

Prune programs deployed in duplicate unconfirmed slot (#32999)

* Prune programs deployed in duplicate unconfirmed slot

* unit test

(cherry picked from commit dbe4017)

Co-authored-by: Pankaj Garg <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v1.16 PRs that should be backported to v1.16
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants