Skip to content

Implement chunked transfer#1048

Open
midenotch wants to merge 3 commits intodotandev:mainfrom
midenotch:implement-chunked-transfer
Open

Implement chunked transfer#1048
midenotch wants to merge 3 commits intodotandev:mainfrom
midenotch:implement-chunked-transfer

Conversation

@midenotch
Copy link
Copy Markdown
Contributor

PULL REQUEST TEMPLATE

================================================================================
TITLE:

Implement chunked transfer for large snapshot arrays - Issue #995

================================================================================
DESCRIPTION:
Implemented rewind support by adding SimulationContext::rollback_to(...) in context.rs and extending the host wrapper in runner.rs. In this checkout the host wrapper lives in runner.rs, so that’s the file I updated instead of the ticket’s host.rs.

Overview

Rollback now rebuilds a fresh Soroban Host from the snapshot’s ledger entries, truncates the context’s event history to the snapshot boundary, and resets host-side events so execution can continue deterministically from that point. I also exported the new modules from lib.rs.

Verification

Verified with:

cargo check --lib
cargo test --lib rollback_to_restores_exact_snapshot_and_truncates_future_events -- --nocapture
cargo test --lib test_restore_from_snapshot_replaces_mutated_storage_and_clears_host_events -- --nocapture

Related Issues

Closes #995

Type of Change

  • New feature
  • Bug fix
  • Breaking change
  • Documentation update

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Tests added/updated
  • Documentation updated
  • No new linting issues
  • Changes verified locally

================================================================================

@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 27, 2026

@midenotch Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@midenotch
Copy link
Copy Markdown
Contributor Author

Done, Close: #1048

@dotandev
Copy link
Copy Markdown
Owner

resolve ci and conflicts.

@midenotch
Copy link
Copy Markdown
Contributor Author

resolve ci and conflicts.

Done

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.

[TIME-TRAVEL] [FEAT] Implement chunked transfer for large snapshot arrays

3 participants