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.

Sign up for GitHub

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

Jump to bottom

[ISSUE #131] Support Snapshot for Dledger #197

Merged
merged 1 commit into from
Oct 10, 2022
Merged

[ISSUE #131] Support Snapshot for Dledger #197

merged 1 commit into from
Oct 10, 2022

Conversation

tsunghanjacktsai
Copy link
Contributor

@tsunghanjacktsai tsunghanjacktsai commented Aug 10, 2022 •
edited
Loading

Test List

  • Unit testing, functional testing of each interface.
  • Test loading snapshot after node restart.
  • Test starting the DLedger under normal mode without a state machine.
  • Test if the number of reserved snapshots meets the specified requirements.
  • Test if previous snapshots can be loaded once the current one fails.

@tsunghanjacktsai tsunghanjacktsai changed the title Support DLedger for Snapshots (temp version) Support DLedger for Snapshots (Temp) Aug 10, 2022
@tsunghanjacktsai tsunghanjacktsai changed the title Support DLedger for Snapshots (Temp) Temp PR Aug 10, 2022
@tsunghanjacktsai tsunghanjacktsai force-pushed the snapshot_tsunghanjacktsai branch 6 times, most recently from c80d0c1 to 8ffeb54 Compare August 17, 2022 07:44
@tsunghanjacktsai tsunghanjacktsai force-pushed the snapshot_tsunghanjacktsai branch 4 times, most recently from 055a80c to 35080b0 Compare August 26, 2022 07:15
@tsunghanjacktsai tsunghanjacktsai force-pushed the snapshot_tsunghanjacktsai branch 5 times, most recently from f6ab81a to db62d2a Compare August 31, 2022 15:44
@RongtongJin RongtongJin changed the title Temp PR [ISSUE #131] Support Snapshot for Dledger Sep 1, 2022
@tsunghanjacktsai tsunghanjacktsai force-pushed the snapshot_tsunghanjacktsai branch from db62d2a to ab9e2ad Compare September 1, 2022 06:04
@RongtongJin
Copy link
Contributor

Hi, @hzh0425 @mxsm @TheR1sing3un Plz help review this PR if you have time.

@tsunghanjacktsai tsunghanjacktsai force-pushed the snapshot_tsunghanjacktsai branch 6 times, most recently from 7acd5b4 to ed3a501 Compare September 4, 2022 14:07
@tsunghanjacktsai tsunghanjacktsai force-pushed the snapshot_tsunghanjacktsai branch from ed3a501 to 4e547bd Compare September 6, 2022 06:36
@tsunghanjacktsai tsunghanjacktsai force-pushed the snapshot_tsunghanjacktsai branch from 4e547bd to 65d0bbd Compare September 6, 2022 09:14
@TheR1sing3un
Copy link
Contributor

TheR1sing3un commented Sep 10, 2022 •
edited
Loading

Hi~ Please resolve the conflicts if you have time~

@tsunghanjacktsai tsunghanjacktsai force-pushed the snapshot_tsunghanjacktsai branch 5 times, most recently from 5ecf86a to f812df1 Compare September 15, 2022 02:45
@tsunghanjacktsai tsunghanjacktsai force-pushed the snapshot_tsunghanjacktsai branch 5 times, most recently from 18e2df5 to bf7bd6c Compare September 20, 2022 02:26
@tsunghanjacktsai tsunghanjacktsai force-pushed the snapshot_tsunghanjacktsai branch 5 times, most recently from f37c4fe to e838c37 Compare September 25, 2022 16:01
@tsunghanjacktsai tsunghanjacktsai force-pushed the snapshot_tsunghanjacktsai branch from e838c37 to 0f5a5d4 Compare September 26, 2022 01:41
@RongtongJin RongtongJin merged commit f00c488 into openmessaging:master Oct 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RongtongJin RongtongJin RongtongJin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tsunghanjacktsai @RongtongJin @TheR1sing3un