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

Update gateway to use DB transactions for Plan Import #1577

Open
dandelany opened this issue Oct 3, 2024 · 0 comments
Open

Update gateway to use DB transactions for Plan Import #1577

dandelany opened this issue Oct 3, 2024 · 0 comments
Assignees
Labels
feature A new feature or feature request gateway Any issues related to the Aerie Gateway service refactor A code change that neither fixes a bug nor adds a feature

Comments

@dandelany
Copy link
Collaborator

dandelany commented Oct 3, 2024

Background

Low-priority follow-up to NASA-AMMOS/aerie-gateway#109 - this PR includes a “faux transaction” ie. our code rolls back DB state if anything goes wrong during import to avoid corruption. This is theoretically almost as safe as a DB transaction, unless gateway dies/errors halfway through.

Description

We'd like to refactor the gateway to use "real" DB transactions for Plan Import (& potentially other similar operations). However this is complicated by two things:

@dandelany dandelany added refactor A code change that neither fixes a bug nor adds a feature feature A new feature or feature request gateway Any issues related to the Aerie Gateway service labels Oct 3, 2024
@github-project-automation github-project-automation bot moved this to Todo in Aerie Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new feature or feature request gateway Any issues related to the Aerie Gateway service refactor A code change that neither fixes a bug nor adds a feature
Projects
Status: Todo
Development

No branches or pull requests

2 participants