Skip to content

Conversation

@giskook
Copy link
Contributor

@giskook giskook commented Mar 29, 2023

change dependency

use customed function to new prunner

add wg.done

add log

format log

add prune command

add human readable logs

add go mod and rm useless lines

  • Targeted PR against correct branch (see CONTRIBUTING.md)

  • Linked to github-issue with discussion and accepted design OR link to spec that describes this work.

  • Wrote tests

  • Updated relevant documentation (docs/)

  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md

  • Reviewed Files changed in the github PR explorer


For Admin Use:

  • Added appropriate labels to PR (ex. wip, ready-for-review, docs)
  • Reviewers Assigned
  • Squashed all commits, uses message "Merge PR #XYZ: [title]" (coding standards)

@giskook giskook requested review from BananaLF, cwbhhjl and xzav3r March 29, 2023 00:29
@giskook giskook marked this pull request as draft March 29, 2023 00:30
@giskook giskook force-pushed the zhangkai/mpt-pruning branch from a4f9e90 to 64640fc Compare March 29, 2023 02:13
change dependency

use customed function to new prunner

add wg.done

add log

format log

add prune command

add human readable logs

add go mod and rm useless lines

delete useless temp code

use the common account related function
@giskook giskook force-pushed the zhangkai/mpt-pruning branch from 0f38fad to 01b58ba Compare March 29, 2023 09:58
@giskook giskook requested a review from lyh169 March 30, 2023 08:09
@KamiD KamiD marked this pull request as ready for review March 31, 2023 08:28
BananaLF
BananaLF previously approved these changes Mar 31, 2023
@KamiD KamiD merged commit 6efeaf2 into dev Apr 4, 2023
giskook added a commit that referenced this pull request May 23, 2023
* migrate go-ethereum's prune command

change dependency

use customed function to new prunner

add wg.done

add log

format log

add prune command

add human readable logs

add go mod and rm useless lines

delete useless temp code

use the common account related function

* change denpendency and if not eth account get code hash panic

* add baseAccount should not panic

---------

Co-authored-by: KamiD <[email protected]>
Co-authored-by: BananaLF <[email protected]>
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.

4 participants