-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
feat(engine, snapshot): snapshot hook #4690
Conversation
Codecov Report
... and 6 files with indirect coverage changes
Flags with carried forward coverage won't be shown. Click here to find out more.
|
a3b7df6
to
d7cdaaf
Compare
d7cdaaf
to
7d86f9d
Compare
177e61e
to
ae0bab6
Compare
5e6f81d
to
48d6246
Compare
48d6246
to
f63dc8e
Compare
64dde70
to
1083908
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
style nits
2675f81
to
fecbfb7
Compare
Ref #4588
This PR adds (but doesn't enable yet) an engine hook for snapshotting and introduces a
snapshot
crate for everything related to creation of snapshots, i.e. theSnapshotter
routine that will be triggered bySnapshotHook
from the engine.