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

Cp accum per epoch component #18442

Merged

Commits on Jun 27, 2024

  1. [core] Make proper per epoch execution components (#18281)

    ## Description 
    
    Make state accumulator a per epoch component. This also requires that we
    make checkpoint executor a proper per epoch component (previously it was
    only instantiated once, but `run_epoch` called once per epoch) since it
    needs to have a reference to the fresh accumulator after reconfig. Now
    we actually drop it after the call to `run_epoch` returns.
    
    ## Test plan 
    
    Passed against 120+ seeds:
    ```
    ./scripts/simtest/seed-search.py simtest --test test_simulated_load_reconfig_with_crashes_and_delays
    ```
    
    ---
    
    ## Release notes
    
    Check each box that your changes affect. If none of the boxes relate to
    your changes, release notes aren't required.
    
    For each box you select, include information after the relevant heading
    that describes the impact of your changes that a user might notice and
    any actions they must take to implement updates.
    
    - [ ] Protocol: 
    - [ ] Nodes (Validators and Full nodes): 
    - [ ] Indexer: 
    - [ ] JSON-RPC: 
    - [ ] GraphQL: 
    - [ ] CLI: 
    - [ ] Rust SDK:
    
    ---------
    
    Co-authored-by: Mark Logan <mark@mystenlabs.com>
    williampsmith and mystenmark committed Jun 27, 2024
    Configuration menu
    Copy the full SHA
    36e3652 View commit details
    Browse the repository at this point in the history