Skip to content

refactor: use gobuffalo/pop for SQL abstraction (#2059)#2129

Merged
aeneasr merged 36 commits intomasterfrom
sql-next
Oct 20, 2020
Merged

refactor: use gobuffalo/pop for SQL abstraction (#2059)#2129
aeneasr merged 36 commits intomasterfrom
sql-next

Conversation

@aeneasr
Copy link
Member

@aeneasr aeneasr commented Oct 12, 2020

This patch replaces the existing SQL and memory managers with a pop based persister. Existing SQL migrations are compatible as they have been migrated to the new SQL abstraction in version 1.7.x. As a goodie, ORY Hydra now supports SQLite for both in-memory as well as on-disk (useful for development and very small deployments) databases!

Closes #1730

Co-authored-by: aeneasr aeneas@ory.sh
Co-authored-by: hackerman 3372410+aeneasr@users.noreply.github.com

To do

  • SQLite currently has no auto_increment/serial for it's PKs

zepatrik and others added 28 commits October 19, 2020 15:54
This patch replaces the existing SQL and memory managers with a pop based persister. Existing SQL migrations are compatible as they have been migrated to the new SQL abstraction in version 1.7.x. As a goodie, ORY Hydra now supports SQLite for both in-memory as well as on-disk (useful for development and very small deployments) databases!

Closes #1730

Co-authored-by: aeneasr <aeneas@ory.sh>
Co-authored-by: hackerman <3372410+aeneasr@users.noreply.github.com>
@aeneasr aeneasr merged commit 1df50ec into master Oct 20, 2020
@aeneasr aeneasr deleted the sql-next branch October 20, 2020 12:55
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.

Migrate to gobuffalo/pop

2 participants