Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Consolidate ledger serialization code #517

Closed
garious opened this issue Jul 2, 2018 · 0 comments
Closed

Consolidate ledger serialization code #517

garious opened this issue Jul 2, 2018 · 0 comments
Assignees
Milestone

Comments

@garious
Copy link
Contributor

garious commented Jul 2, 2018

Move the serde_json::from_str code from bin/fullnode.rs and bank.rs into entry_writer.rs under new static methods read_entry() and read_entries().

@garious garious added this to the The Future! milestone Jul 2, 2018
garious added a commit to garious/solana that referenced this issue Jul 2, 2018
The new read_entries() works, but is overly-contrained. Not
using that function yet, but adding it here in the hopes some
Rust guru will tell us how to get that lifetime constraint out
of there.

Fixes solana-labs#517
@rob-solana rob-solana self-assigned this Jul 2, 2018
@garious garious closed this as completed in ec7e50b Jul 2, 2018
@garious garious modified the milestones: The Future!, v0.7.0 Jul 18, 2018
vkomenda pushed a commit to vkomenda/solana that referenced this issue Aug 29, 2021
CriesofCarrots pushed a commit to CriesofCarrots/solana that referenced this issue Apr 1, 2024
* build(deps): bump serde_json from 1.0.114 to 1.0.115

Bumps [serde_json](https://github.com/serde-rs/json) from 1.0.114 to 1.0.115.
- [Release notes](https://github.com/serde-rs/json/releases)
- [Commits](serde-rs/json@v1.0.114...v1.0.115)

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

Signed-off-by: dependabot[bot] <support@github.com>

* [auto-commit] Update all Cargo lock files

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot-buildkite <dependabot-buildkite@noreply.solana.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants