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

New exec code path rebased with real impl #15

Merged

Conversation

ryoqun
Copy link
Owner

@ryoqun ryoqun commented Nov 17, 2023

Problem

Summary of Changes

Fixes #

Tyera and others added 28 commits December 13, 2023 00:16
* Patch tokio to vendored version

* Fix typo
* build(deps): bump syn from 2.0.39 to 2.0.40

Bumps [syn](https://github.com/dtolnay/syn) from 2.0.39 to 2.0.40.
- [Release notes](https://github.com/dtolnay/syn/releases)
- [Commits](dtolnay/syn@2.0.39...2.0.40)

---
updated-dependencies:
- dependency-name: syn
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* [auto-commit] Update all Cargo lock files

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot-buildkite <[email protected]>
* build(deps): bump rustls from 0.21.9 to 0.21.10

Bumps [rustls](https://github.com/rustls/rustls) from 0.21.9 to 0.21.10.
- [Release notes](https://github.com/rustls/rustls/releases)
- [Changelog](https://github.com/rustls/rustls/blob/main/CHANGELOG.md)
- [Commits](rustls/rustls@v/0.21.9...v/0.21.10)

---
updated-dependencies:
- dependency-name: rustls
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* [auto-commit] Update all Cargo lock files

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot-buildkite <[email protected]>
* build(deps): bump aquamarine from 0.3.2 to 0.3.3

Bumps [aquamarine](https://github.com/mersinvald/aquamarine) from 0.3.2 to 0.3.3.
- [Release notes](https://github.com/mersinvald/aquamarine/releases)
- [Changelog](https://github.com/mersinvald/aquamarine/blob/master/CHANGELOG.md)
- [Commits](https://github.com/mersinvald/aquamarine/commits)

---
updated-dependencies:
- dependency-name: aquamarine
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* [auto-commit] Update all Cargo lock files

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot-buildkite <[email protected]>
…olana-labs#34426)

load_and_process_ledger() performs many checks and sub-operations that
can fail. The current error handling prints an error message and exits
immediately. The long error/help messages written inline add clutter
to the functions actual implementation.

This PR creates a new error enum for all of these previous error
conditions, and bubbles up the error to let the caller decide what to
do instead of exiting immediately.
…mgSbA7mjpJFgX5zP2dkp8qsF2C33V (solana-labs#34089)

cleanup feature code after mainnet-beta activation
Several commands call load_and_process_ledger() which can fail in a
number of ways. These callers currently all handle the result in the
same way by matching the return Result:
- The Ok(_) case uses the returned types as normal
- The Err(_) case prints an error message and exits

This error handling is redundant, and a helper could remove the
duplicate code. So, this PR adds a wrapper around that checks the
result and unwraps OR prints error messages and exits.
…lana-labs#34448)

The same initialization occurs when the Bank is inserted into a new
BankForks, so no need to duplicate the logic in this test function
@ryoqun ryoqun merged commit c0292a9 into new-exec-code-path-rebased Dec 20, 2023
2 checks passed
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.