Skip to content

Flush callbacks one more time at shutdown#3963

Merged
PastaPastaPasta merged 1 commit into
dashpay:developfrom
UdjinM6:fix_callbacks_flush_on_shutdown
Jan 25, 2021
Merged

Flush callbacks one more time at shutdown#3963
PastaPastaPasta merged 1 commit into
dashpay:developfrom
UdjinM6:fix_callbacks_flush_on_shutdown

Conversation

@UdjinM6

@UdjinM6 UdjinM6 commented Jan 22, 2021

Copy link
Copy Markdown

This is needed to be able to process SetBestChain callback generated by the first FlushStateToDisk call in PrepareShutdown. Not doing this leaves the wallet in an inconsistent state which is one of the reasons why feature_pruning.py fails atm.

Partially reverts #3378

This is needed to be able to process SetBestChain callback generated by the first FlushStateToDisk call in PrepareShutdown

Partially reverts dashpay#3378
@UdjinM6 UdjinM6 added the bug label Jan 22, 2021
@UdjinM6 UdjinM6 added this to the 17 milestone Jan 22, 2021
@UdjinM6
UdjinM6 marked this pull request as ready for review January 22, 2021 18:52

@xdustinface xdustinface left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@PastaPastaPasta PastaPastaPasta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@PastaPastaPasta
PastaPastaPasta merged commit a328756 into dashpay:develop Jan 25, 2021
@UdjinM6
UdjinM6 deleted the fix_callbacks_flush_on_shutdown branch July 1, 2021 21:56
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Apr 1, 2022
This is needed to be able to process SetBestChain callback generated by the first FlushStateToDisk call in PrepareShutdown

Partially reverts dashpay#3378
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants