feat: allow SQLite database backend #1663
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…sages fix deleting model remove unrelated changes Update GameMessages.cpp
|
update migration runner, add back mysql files then we're done |
Xiphoseer
previously approved these changes
Dec 10, 2024
aronwk-aaron
previously approved these changes
Dec 12, 2024
aronwk-aaron
previously approved these changes
Dec 17, 2024
jadebenn
reviewed
Dec 17, 2024
Xiphoseer
previously approved these changes
Dec 17, 2024
EmosewaMC
dismissed stale reviews from Jettford, Xiphoseer, jadebenn, and aronwk-aaron
via
e3c9c71
EmosewaMC
requested review from
Xiphoseer,
Jettford,
jadebenn and
aronwk-aaron
Xiphoseer
approved these changes
Dec 17, 2024
aronwk-aaron
approved these changes
Dec 17, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds SQLite as a backend database for server data.
Updates various components to work with the new file structure and migration setup.
Consolidate migations into 1 initial one for sqlite.
Adds migration for mysql migrations to migrate them to the new filepath.
Ensure backwards compatibility with MariaDB setups.
Ensure default from now on is SQLite.
Update README with the new setup instructions.
Send exes from subprocesses to the background so players are not interrupted in gameplay by a new window every world transfer.
Adds CppSQLite3 binds for sqlite_int64 types.
New migrations are moved and placed in the correct directories.
Tested that all queries are functional as before and that gameplay is unchanged.
There are known issues with windows and how it handles shutting down the server which I will address in a future PR as I already know of a fix for this for single player servers specifically.