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

--wip-- architect and migration #30190

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

kaapstorm
Copy link
Contributor

Summary

Draft PR of implementation of CEP 30189. Opening for the sake of illustration and discussion. Not for merging.

@kaapstorm kaapstorm added the product/invisible Change has no end-user visible impact label Aug 3, 2021
@dimagimon dimagimon added the reindex/migration Reindex or migration will be required during or before deploy label Aug 3, 2021
def get_sql():
next_month = '2021-09-01 00:00:00 Z' # TODO: Calculate this
return f"""
CREATE TABLE motech_requestlog (
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if this table has exactly the same schema as the existing one you could say CREATE TABLE motech_requestlog LIKE dhis2_jsonapilog INCLUDING ALL

https://www.postgresql.org/docs/9.6/sql-createtable.html

@kaapstorm
Copy link
Contributor Author

NOTE TO SELF: This migration does not reverse

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product/invisible Change has no end-user visible impact reindex/migration Reindex or migration will be required during or before deploy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants