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 harvest DB tables #4731

Closed
1 task
rshewitt opened this issue May 6, 2024 · 3 comments
Closed
1 task

Update harvest DB tables #4731

rshewitt opened this issue May 6, 2024 · 3 comments
Assignees
Labels
H2.0/Harvest-DB Postgres and related DB tickets

Comments

@rshewitt
Copy link
Contributor

rshewitt commented May 6, 2024

User Story

In order to use the most recent table definitions, datagov wants to update the harvest database tables to reflect what's in the table definition document

Acceptance Criteria

[ACs should be clearly demoable/verifiable whenever possible. Try specifying them using BDD.]

  • GIVEN the existing db models in harvesting logic
    WHEN an update to the model happens
    THEN the new table(s) should exist as they are in the table definition file \

Background

[Any helpful contextual notes or links to artifacts/evidence, if needed]

Security Considerations (required)

[Any security concerns that might be implicated in the change. "None" is OK, just be explicit here!]

Sketch

[Notes or a checklist reflecting our understanding of the selected approach]

@rshewitt rshewitt added the H2.0/Harvest-DB Postgres and related DB tickets label May 6, 2024
@rshewitt rshewitt self-assigned this May 13, 2024
@rshewitt
Copy link
Contributor Author

needs to occur before #4744

@rshewitt
Copy link
Contributor Author

columns are nullable by default (source). just wanted to share

@rshewitt
Copy link
Contributor Author

a conclusion that we reached in our stay-after is to drop sqlite and look into scaling another instance of postgres. one for the flask app and another one for db integration tests. we also discussed moving the flask app out of the compose file and into the makefile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
H2.0/Harvest-DB Postgres and related DB tickets
Projects
Status: 🗄 Closed
Development

No branches or pull requests

2 participants