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.

Sign up for GitHub

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

Jump to bottom

Remove archive block handling #1580

Closed
beckjake opened this issue Jun 28, 2019 · 1 comment
Closed

Remove archive block handling #1580

beckjake opened this issue Jun 28, 2019 · 1 comment
Milestone
Louisa May Alcott

Comments

@beckjake
Copy link
Contributor

Feature

Remove support for the archive block (and therefore remove the snapshot migration task)

Feature description

In Wilt Chamberlain, we removed support for archives and rename them to "snapshots". As part of that, we disabled support for the "archive" configuration, but kept configuration-level support if a special flag is passed. That flag is used exclusively by the migration task.

In Louisa May Alcott we should remove that parser support for real, which entails removing the task. It will mean that anyone upgrading from <0.14.x to >=0.15.x will have to stop at 0.14.x and run the migration script. That seems fair to me!

Who will this benefit?

Me - it's one less thing to convert to dataclasses.
@beckjake beckjake added this to the Louisa May Alcott milestone Jun 28, 2019
@beckjake beckjake mentioned this issue Jun 28, 2019
Merged
beckjake added a commit that referenced this issue Jun 28, 2019
@beckjake
Merge pull request #1581 from fishtown-analytics/feature/remove-archi…
b9bfff1 
…ve-config

remove archive block support and "dbt snapshot-migrate" (#1580)
@drewbanin
Copy link
Contributor

Closed by #1581

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Louisa May Alcott
Development

No branches or pull requests
2 participants
@beckjake @drewbanin