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

[3.2] Implement JSON Snapshot Reader #23

Closed
wants to merge 10 commits into from

Conversation

ClaytonCalabrese
Copy link
Contributor

@ClaytonCalabrese ClaytonCalabrese commented Aug 18, 2022

eosnetworkfoundation/mandel#710 ports a way to convert a snapshot to json. This PR enhances the --snapshot option to support reading a snapshot in JSON format if the snapshot has .json extension.

Resolves: eosnetworkfoundation/mandel#771

Previous discussion: eosnetworkfoundation/mandel#732

Currently working on test, then will bring out of draft.

heifner and others added 10 commits July 20, 2022 11:21
…erge snapshot and jsonsnapshot code handling
…d of adding a new option. Improved memory usage so current WAX snapshot can be supported. Modified the JSON output so that table rows are an array which reduces file size and memory usage. Put rapidjson inside an eosio_rapidjson namespace to avoid ODR issues. Change CMakeLists to not compile rapidjson but only include it as it is a header only library.
@ClaytonCalabrese ClaytonCalabrese added the OCI Work exclusive to OCI team label Aug 18, 2022
@ClaytonCalabrese ClaytonCalabrese self-assigned this Aug 18, 2022
@ClaytonCalabrese ClaytonCalabrese marked this pull request as draft August 18, 2022 16:14
@ClaytonCalabrese ClaytonCalabrese changed the title [3.2] Add command line option --snapshot-to-json [3.2] Implement JSON Snapshot Reader Aug 18, 2022
greg7mdp added a commit that referenced this pull request Dec 15, 2023
131/150 Test   #98: producer_schedule_hs_unit_test_eos-vm-jit ..........***Failed    1.36 sec
132/150 Test   #97: producer_schedule_hs_unit_test_eos-vm ..............***Failed    1.57 sec
133/150 Test   #96: producer_schedule_hs_unit_test_eos-vm-oc ...........***Failed    2.86 sec
144/150 Test   #21: api_unit_test_eos-vm-oc ............................***Failed   79.26 sec
146/150 Test   #23: api_unit_test_eos-vm-jit ...........................***Failed   96.61 sec
150/150 Test   #22: api_unit_test_eos-vm ...............................***Failed  467.14 sec
heifner added a commit that referenced this pull request Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCI Work exclusive to OCI team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cleanup and improve json snapshot support
2 participants