Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Extract state-db from ethcore #10858

Merged
merged 78 commits into from
Jul 8, 2019
Merged

Extract state-db from ethcore #10858

merged 78 commits into from
Jul 8, 2019

Conversation

dvdplm
Copy link
Collaborator

@dvdplm dvdplm commented Jul 8, 2019

Builds on #10840 – relevant commits are the last three and especially 40aef44

Moves ethcore/state_db to its own crate. Reduces the public API somewhat.

dvdplm added 30 commits July 3, 2019 15:57
Remove botched ethcore-error crate
* master:
  Extricate PodAccount and state Account to own crates (#10838)
  logs (#10817)
  refactor: whisper: Add type aliases and update rustdocs in message.rs (#10812)
  Break circular dependency between Client and Engine (part 1) (#10833)
  tests: Relates to #10655: Test instructions for Readme (#10835)
  refactor: Related #9459 - evmbin: replace untyped json! macro with fully typed serde serialization using Rust structs (#10657)
…hore/extricate-state-backend

* dp/chore/extricate-account-db-into-own-crate:
  third time's the charm
  test failure 2
  test failure
  Extricate PodAccount and state Account to own crates (#10838)
  logs (#10817)
  refactor: whisper: Add type aliases and update rustdocs in message.rs (#10812)
  Break circular dependency between Client and Engine (part 1) (#10833)
  tests: Relates to #10655: Test instructions for Readme (#10835)
  refactor: Related #9459 - evmbin: replace untyped json! macro with fully typed serde serialization using Rust structs (#10657)
* master:
  Extract AccountDB to account-db (#10839)
  test: Update Whisper test for invalid pool size (#10811)
* master:
  Improve logging and cleanup in miner around block sealing (#10745)
@dvdplm dvdplm changed the title Builds on https://github.com/paritytech/parity-ethereum/pull/10840 Extract state-db from ethcore Jul 8, 2019
@dvdplm dvdplm self-assigned this Jul 8, 2019
@dvdplm dvdplm added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Jul 8, 2019
@dvdplm dvdplm added this to the 2.7 milestone Jul 8, 2019
@dvdplm dvdplm requested review from ordian and seunlanlege July 8, 2019 09:59
@seunlanlege
Copy link
Member

@dvdplm please rebase, reviewing this will take a while

* master:
  EIP-1702: Generalized Account Versioning Scheme (#10771)
…-state-db

* dp/chore/extricate-state-backend:
  Sort out the merge
  EIP-1702: Generalized Account Versioning Scheme (#10771)
@dvdplm
Copy link
Collaborator Author

dvdplm commented Jul 8, 2019

@dvdplm please rebase, reviewing this will take a while

It's really only 40aef44 that is relevant, but it'd be fantastic to get a review on the full #10840 as well! :)

@ordian
Copy link
Collaborator

ordian commented Jul 8, 2019

the 40aef44 lgtm, although having a circular dependency for tests is unfortunate
Will approve after #10840 is merged and this PR is rebased.

* master:
  Move more code into state-account (#10840)
  Remove compiler warning (#10865)
  [ethash] use static_assertions crate (#10860)
@dvdplm
Copy link
Collaborator Author

dvdplm commented Jul 8, 2019

@seunlanlege @ordian ptal

@dvdplm dvdplm added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jul 8, 2019
@seunlanlege seunlanlege merged commit 0f33717 into master Jul 8, 2019
@seunlanlege seunlanlege deleted the dp/chore/extract-state-db branch July 8, 2019 21:01
ordian added a commit that referenced this pull request Jul 9, 2019
* master:
  Run cargo fix on a few of the worst offenders (#10854)
  removed redundant fork choice abstraction (#10849)
  Extract state-db from ethcore (#10858)
  Fix fork choice (#10837)
  Move more code into state-account (#10840)
  Remove compiler warning (#10865)
  [ethash] use static_assertions crate (#10860)
dvdplm added a commit that referenced this pull request Jul 9, 2019
* master:
  Run cargo fix on a few of the worst offenders (#10854)
  removed redundant fork choice abstraction (#10849)
  Extract state-db from ethcore (#10858)
  Fix fork choice (#10837)
  Move more code into state-account (#10840)
  Remove compiler warning (#10865)
  [ethash] use static_assertions crate (#10860)
dvdplm added a commit that referenced this pull request Jul 11, 2019
* master:
  Docker images renaming (#10863)
  Move the substate module into ethcore/executive (#10867)
  Run cargo fix on a few of the worst offenders (#10854)
  removed redundant fork choice abstraction (#10849)
  Extract state-db from ethcore (#10858)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants