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

SQLite Storage Option #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

SQLite Storage Option #14

wants to merge 2 commits into from

Conversation

Kyrobi
Copy link

@Kyrobi Kyrobi commented Oct 24, 2024

Implements another option for storing the data.

Currently, one of the pain points of flat files is that FTP/SFTP transfer of the files take a long time. When moving server files around, it's often that transferring the data takes up a significant portion of the time.

However, with MySQL most shared providers have limited amount of SQL databases you can create, so if existing plugins already require it, then you don't be about to use the MySQL storage. Additionally, it would be nice to have a self-contained storage like SQLite instead of having to rely on a separate database server.

Implements another option for storing the data.

Currently, one of the pain points of flat files is that FTP/SFTP transfer of the files take a long time. When moving server files around, it's often that transferring the data takes up a significant portion of the time.

However, with MySQL most shared providers have limited amount of SQL databases you can create, so if existing plugins already require it, then you don't be about to use the MySQL storage. Additionally, it would be nice to have a self-contained storage like SQLite instead of having to rely on a separate database server.
The plugin still makes the http call to check for the update even while the update checker setting is turned off. It just won't display the result.

Moved the check off the main thread or else the server will stall when there's something wrong with the http connection. If there is slow connection or the spigot website is down, the server will stall or crash.
@Kyrobi
Copy link
Author

Kyrobi commented Oct 30, 2024

Oops, should have opened a new branch

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.

1 participant