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
Auto-snapshot management #9739
Auto-snapshot management #9739
Conversation
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
hedera-node/test-clients/record-snapshots/CryptoTransfer-SpecialAccountsBalanceCheck.json
Outdated
Show resolved
Hide resolved
...st-clients/src/main/java/com/hedera/services/bdd/suites/crypto/AutoAccountCreationSuite.java
Outdated
Show resolved
Hide resolved
...st-clients/src/main/java/com/hedera/services/bdd/suites/crypto/AutoAccountCreationSuite.java
Outdated
Show resolved
Hide resolved
...t-clients/src/main/java/com/hedera/services/bdd/spec/utilops/records/AutoSnapshotModeOp.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
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.
LGTM ! Thanks @tinker-michaelj 🙌
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.
LGTM
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
730c0c4
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
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.
Looks Great 🎉 . Thanks @tinker-michaelj
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com> Signed-off-by: Ivo Yankov <ivo@devlabs.bg>
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com> Signed-off-by: Nick Poorman <nick@swirldslabs.com>
Description:
HapiSpec
snapshot matching #9697HapiSpec
JSON snapshots into a single per-HapiSuite
snapshot.AutoSnapshotModeOp
that delegates to aSnapshotModeOp
with a mode determined by:HapiSpec
to insert anAutoSnapshotModeOp
at the start of the spec ifrecordStream.autoSnapshotManagement=true
.CryptoTransferSuite
.HapiCryptoTransfer
transfer list construction deterministic by sorting.