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

[Suggestion (Plugin)]: Database Refactoring & Preparing for various features #512

Closed
sarhatabaot opened this issue Dec 9, 2024 · 0 comments · Fixed by #541
Closed

[Suggestion (Plugin)]: Database Refactoring & Preparing for various features #512

sarhatabaot opened this issue Dec 9, 2024 · 0 comments · Fixed by #541
Assignees
Labels
optimizations (readability) Readability improvements suggestion: plugin A suggestion for the plugin.
Milestone

Comments

@sarhatabaot
Copy link
Collaborator

Suggestion

We should refactor the database class to achieve a couple of objectives:

  1. More databases support, in addition to the ones already supported (MySQL, SQLite): MariaDB, PostreSQL, H2, HyperSQL
  2. Reduce amount of raw SQL used and use some form of system instead. (Hibernate, jOOQ, Jdbi, Exposed etc)
  3. Improve migration system with flyway.
  4. The database should be enabled by default with a local (SQLite, H2, HyperSQL) database, to allow for various features ([Suggestion (Plugin)]: Fishing Journal #371, Fish caught history & all time tracking #264, Suggestion: overall Leaderboard Fish/Users #121).

If I have more ideas I'll write them here

@sarhatabaot sarhatabaot added suggestion: plugin A suggestion for the plugin. optimizations (readability) Readability improvements labels Dec 9, 2024
@sarhatabaot sarhatabaot self-assigned this Dec 9, 2024
@FireML FireML mentioned this issue Dec 16, 2024
13 tasks
@sarhatabaot sarhatabaot added this to the 2.0.0 milestone Feb 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
optimizations (readability) Readability improvements suggestion: plugin A suggestion for the plugin.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant