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

Hotfix/Delay backup until after initialization. #4281

Merged
merged 2 commits into from Aug 21, 2020
Merged

Hotfix/Delay backup until after initialization. #4281

merged 2 commits into from Aug 21, 2020

Conversation

ghost
Copy link

@ghost ghost commented Aug 21, 2020

Description:

This simply delays the backup task until all db initializations including migrations have occurred.


  • I have tested this pull request for defects on a server.

By making this pull request, I represent that I have the right to waive copyright and related rights to my contribution, and agree that all copyright and related rights in my contributions are waived, and I acknowledge that the TownyAdvanced organization has the copyright to use and modify my contribution under the Towny License for perpetuity.

@LlmDl LlmDl merged commit a5b0677 into TownyAdvanced:master Aug 21, 2020
LlmDl added a commit that referenced this pull request Aug 22, 2020
with PR #4281.
  - Readd TownyMessaging.sendMsg(Object, String) deprecated.
@LlmDl
Copy link
Member

LlmDl commented Aug 23, 2020

It would appear that this PR has caused (at least on SQL servers) lockups to the SQL server, not allowing the players' resident data to be saved when they log in. https://hastebin.com/fayehimapi.md

Perhaps you could look into, otherwise this might have to be reverted. The above errors disappeared when the config's flatfile_backup_type was set to none.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants