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

added recovery logic around ledger entry persistence failures #1505

Merged
merged 4 commits into from Jun 4, 2019

Conversation

Projects
None yet
2 participants
@gaboraranyossy-da
Copy link
Contributor

commented Jun 4, 2019

This PR introduces recovery over failing Postgres persistence operations. Without having this recovery the Sandbox will stop working as soon as a Postgres persistence operation fails.

Pull Request Checklist

NOTE: CI is not automatically run on non-members pull-requests for security
reasons. The reviewer will have to comment with /AzurePipelines run to
trigger the build.

@gaboraranyossy-da gaboraranyossy-da requested a review from gerolf-da Jun 4, 2019

@gaboraranyossy-da gaboraranyossy-da marked this pull request as ready for review Jun 4, 2019

@gaboraranyossy-da gaboraranyossy-da requested a review from bethaitman as a code owner Jun 4, 2019

@gaboraranyossy-da gaboraranyossy-da changed the title added recovery logic when ledger entry persistence fails added recovery logic around ledger entry persistence failures Jun 4, 2019

@gaboraranyossy-da gaboraranyossy-da merged commit 35ef13f into master Jun 4, 2019

7 checks passed

Summary 2 potential rules
Details
digital-asset.daml Build #20190604.65 succeeded
Details
digital-asset.daml (Linux) Linux succeeded
Details
digital-asset.daml (Windows) Windows succeeded
Details
digital-asset.daml (macOS) macOS succeeded
Details
digital-asset.daml (perf) perf succeeded
Details
license/cla Contributor License Agreement is signed.
Details

@gaboraranyossy-da gaboraranyossy-da deleted the persistence-queue-recovery branch Jun 4, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.