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

Add DynamoDB Support #91

Closed
wants to merge 8 commits into from
Closed

Add DynamoDB Support #91

wants to merge 8 commits into from

Conversation

jamiefdhurst
Copy link
Owner

@jamiefdhurst jamiefdhurst commented Apr 30, 2024

Fixes #86

@jamiefdhurst jamiefdhurst marked this pull request as draft April 30, 2024 19:54
@jamiefdhurst jamiefdhurst self-assigned this Apr 30, 2024
Copy link

github-actions bot commented Apr 30, 2024

Test Results

53 tests   53 ✅  28s ⏱️
10 suites   0 💤
 1 files     0 ❌

Results for commit 07d5d09.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 5, 2024

Package Line Rate Health
github.com/jamiefdhurst/journal/internal/app 3%
github.com/jamiefdhurst/journal/internal/app/controller/apiv1 100%
github.com/jamiefdhurst/journal/internal/app/controller/web 89%
github.com/jamiefdhurst/journal/internal/app/model 70%
github.com/jamiefdhurst/journal/internal/app/router 100%
github.com/jamiefdhurst/journal/pkg/adapter/giphy 100%
github.com/jamiefdhurst/journal/pkg/adapter/json 100%
github.com/jamiefdhurst/journal/pkg/controller 100%
github.com/jamiefdhurst/journal/pkg/database/dynamodb 94%
github.com/jamiefdhurst/journal/pkg/database/sqlite 100%
github.com/jamiefdhurst/journal/pkg/router 100%
github.com/jamiefdhurst/journal/pkg/session 83%
Summary 81% (423 / 522)

Minimum allowed line rate is 80%

@jamiefdhurst
Copy link
Owner Author

Closing this, the plan is to move to EFS instead and support Lambdas through a shared filesystem, which requires no re-platforming of the database.

@jamiefdhurst jamiefdhurst deleted the feature/dynamodb branch May 14, 2024 21:10
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.

Alternate DB to SQLite, e.g. DynamoDB
1 participant