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

feat(GODT-2510): Direct SQLite3 #366

Merged
merged 2 commits into from
Jun 13, 2023
Merged

Conversation

LBeernaertProton
Copy link
Contributor

DB implementation using SQLite3. This current version is feature complete with the ent implementation. The only noticeable difference is the addition of the db version table.

The new database is now the default, but we will still run tests on CI to compare behavior against old implementation until it is completely removed.

@LBeernaertProton LBeernaertProton force-pushed the feat/GODT-2510-sqlite branch 2 times, most recently from 35e71bb to 4f5899e Compare June 12, 2023 15:56
internal/db_impl/sqlite3/types.go Outdated Show resolved Hide resolved
internal/db_impl/sqlite3/migration_v0.go Outdated Show resolved Hide resolved
internal/db_impl/sqlite3/client.go Show resolved Hide resolved
@xmichelo xmichelo self-requested a review June 13, 2023 06:54
xmichelo
xmichelo previously approved these changes Jun 13, 2023
DB implementation using SQLite3. This current version is feature
complete with the ent implementation. The only noticeable difference is
the addition of the db version table.

The new database is now the default, but we will still run tests on CI
to compare behavior against old implementation until it is completely
removed.
This test fails a lot on CI due to some events not being processed at
the right time. This will be checked later with GODT-2688.
@LBeernaertProton LBeernaertProton merged commit ee1ced1 into dev Jun 13, 2023
4 checks passed
@LBeernaertProton LBeernaertProton deleted the feat/GODT-2510-sqlite branch June 13, 2023 11:26
@github-actions
Copy link

github-actions bot commented Jul 7, 2023

🎉 This PR is included in version 0.17.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants