Skip to content

Refactor!: Replace snapshot model with snapshot node#1137

Merged
crericha merged 9 commits intomainfrom
snapshot-nodes
Jul 18, 2023
Merged

Refactor!: Replace snapshot model with snapshot node#1137
crericha merged 9 commits intomainfrom
snapshot-nodes

Conversation

@crericha
Copy link
Copy Markdown
Collaborator

@crericha crericha commented Jul 12, 2023

Setting the stage for moving audits to their own nodes.

@crericha crericha requested review from a team, izeigerman and tobymao July 12, 2023 04:37
Comment thread sqlmesh/core/node.py Outdated
Comment thread sqlmesh/core/node.py Outdated
Comment thread sqlmesh/core/node.py Outdated
@crericha crericha force-pushed the snapshot-nodes branch 2 times, most recently from 1272692 to 5f8bc83 Compare July 18, 2023 02:17
Comment thread sqlmesh/core/model/definition.py Outdated
Comment thread sqlmesh/core/node.py Outdated
Comment thread sqlmesh/migrations/v0018_rename_snapshot_model_to_node.py Outdated
Comment thread sqlmesh/migrations/v0018_rename_snapshot_model_to_node.py Outdated
Copy link
Copy Markdown
Collaborator

@izeigerman izeigerman left a comment

Choose a reason for hiding this comment

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

🚢 🤞

Comment thread sqlmesh/core/model/definition.py Outdated
@crericha crericha merged commit 8846ff4 into main Jul 18, 2023
@crericha crericha deleted the snapshot-nodes branch July 18, 2023 22:03
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.

4 participants