Skip to content

[PROPOSAL] remove mysql support#658

Open
parkan wants to merge 1 commit into
mainfrom
remove-mysql-support
Open

[PROPOSAL] remove mysql support#658
parkan wants to merge 1 commit into
mainfrom
remove-mysql-support

Conversation

@parkan
Copy link
Copy Markdown
Collaborator

@parkan parkan commented Apr 10, 2026

maintaining around mysql quirks has been an absolute nightmare and is of unclear value; some of the new features (e.g. shovel) are natively postgres-only

propose removing mysql support to reduce maintainer load

removes:

  • mysql driver (gorm.io/driver/mysql, go-sql-driver/mysql)
  • mysql-specific dialect branches in migrate.go, healthcheck, export-keys
  • mysql deadlock debugging helpers (database/deadlock_debug.go, testutil/deadlock.go)
  • mysql-only cascade cycle test
  • mariadb from devcontainer (Dockerfile, init/start scripts, env vars)
  • mysql setup from CI action
  • mysql from SupportedTestDialects

@parkan parkan force-pushed the remove-mysql-support branch 3 times, most recently from 4f2766e to 54a59c3 Compare April 10, 2026 13:29
@parkan
Copy link
Copy Markdown
Collaborator Author

parkan commented Apr 10, 2026

sorry about the failure noise, github runners are always lots of fun

@parkan parkan marked this pull request as ready for review April 24, 2026 09:15
only sqlite and postgres are supported going forward. operators on mysql
should migrate to postgres before upgrading -- the binary now refuses to
start with a mysql connection string.

removes:
- mysql driver (gorm.io/driver/mysql, go-sql-driver/mysql)
- mysql-specific dialect branches in migrate.go, healthcheck, export-keys
- mysql deadlock debugging helpers (database/deadlock_debug.go, testutil/deadlock.go)
- mysql-only cascade cycle test
- mariadb from devcontainer (Dockerfile, init/start scripts, env vars)
- mysql setup from CI action
- mysql from SupportedTestDialects
@parkan parkan force-pushed the remove-mysql-support branch from 54a59c3 to 0312456 Compare April 24, 2026 10:49
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