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.

Sign up for GitHub

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

Jump to bottom

Migration to PostgreSQL and sqlx version upgrade #39

Merged
merged 32 commits into from
Aug 22, 2024
Merged

Migration to PostgreSQL and sqlx version upgrade #39

merged 32 commits into from
Aug 22, 2024

Conversation

bitfl0wer
Copy link
Member

@bitfl0wer bitfl0wer commented Aug 20, 2024 •
edited
Loading

  • Upgrade sqlx from v0.7.4 to v0.8.0 (latest)
  • Migrate symfonia away from MySQL/MariaDB, over to PostgreSQL by popular demand

Running docker compose up --build with the given composefile and an existing, valid .env file starts the server up without error.

Todos from here:

  • Break out references to other objects into own tables instead of putting them as JSON

bitfl0wer added 23 commits August 4, 2024 17:25
@bitfl0wer
cargo fmt
eb02685
@bitfl0wer
postgres time :3
32ddc65
@bitfl0wer
cargo fmt
138dab1
@bitfl0wer
Several fixes to do with migrating to postgres
7627e11
@bitfl0wer
Several fixes to do with migrating to PostgreSQL
6604f1e
@bitfl0wer
Remove --all from pre-push hook command
31209ac
@bitfl0wer
More PostgreSQL migration fixes
6b82752
@bitfl0wer
Use git for chorus
6c19d6d
@bitfl0wer
Move compose files to postgres
1098840
@bitfl0wer
remove any:: driver initialization
7886c18
@bitfl0wer
Fix: Convert MySQL function call to Postgres
ca78e50
@bitfl0wer
migrations: replace tinyint with smallint
6288f81
@bitfl0wer
migrations: replace datetime with timestamp
f33160c
@bitfl0wer
migrations: replace \key\ with key
2bd2f54
@bitfl0wer
migrations: replace \index\ with index
08c0d4f
@bitfl0wer
migrations: convert unsigned to postgres unsigned representation
ad6ed80
@bitfl0wer
migrations: replace \system\ with system
45e4b3c
@bitfl0wer
migrations: replace \groups\ with groups
0916888
@bitfl0wer
migrations: replace auto_increment with PG serial type
11097e5
@bitfl0wer
migrations: replace mysql statements with pgsql statements
2304e96
@bitfl0wer
Change healthcheck for db to feel more responsive
4112dee
@bitfl0wer
migrations: fix config type text needing to be json instead
c2febf1
@bitfl0wer
Fix "fatal: role root doesn't exist" log spam
f4803f1
github-advanced-security[bot]
github-advanced-security bot found potential problems Aug 20, 2024
View reviewed changes
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

clippy found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@bitfl0wer bitfl0wer requested a review from Quat3rnion August 20, 2024 18:02
Quat3rnion
Quat3rnion reviewed Aug 21, 2024
View reviewed changes
Quat3rnion
Quat3rnion reviewed Aug 21, 2024
View reviewed changes
Copy link
Collaborator

@Quat3rnion Quat3rnion left a comment

Choose a reason for hiding this comment

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

Everything else looks great, and the things I noted aren't explicitly necessary to merge either

@bitfl0wer bitfl0wer merged commit 292c1cd into main Aug 22, 2024
4 checks passed
@bitfl0wer bitfl0wer mentioned this pull request Aug 22, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Quat3rnion Quat3rnion Quat3rnion left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bitfl0wer @Quat3rnion