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

Add snowflake operations #1

Merged
merged 13 commits into from
Feb 10, 2021

Conversation

clrcrl
Copy link
Contributor

@clrcrl clrcrl commented Feb 9, 2021

Sort of a big PR, sorry about this! Feel free to push back and make me chunk it up!

This PR:

  • Adds macros to upload artifacts via an operation (including relevant instructions)
  • Adds integration tests (these run successfully on our version of this repo — if you want me to remove them, happy to do so)
  • Adds default values for the database, schema, and table name for the source table

@NiallRees NiallRees merged commit 0823adf into brooklyn-data:main Feb 10, 2021
NiallRees pushed a commit that referenced this pull request Feb 10, 2021
* Add trailing newlines

* Add Snaptravel's macro

* Add macro to create artifact resources

* Make upload_artifacts work with package

* Add integration tests

* Add trailing newlines

* Use the source more meaningfully

* Make filenames configurable

* Add integration tests for source freshness

* Update README

* Refactor macro

* Add require-dbt-version config

* Allow .gz extension
alanmcruickshank pushed a commit that referenced this pull request Nov 29, 2021
alanmcruickshank pushed a commit that referenced this pull request Nov 29, 2021
* Add seed, snapshot, test dimensions and executions

* Change run_results to include seed, snapshot, test

* Update README with models

* Update attribute names to be resource-specific

* Update resource columns in execution models

* Pass depends_on_nodes to fct model
NiallRees added a commit that referenced this pull request Feb 15, 2022
NiallRees added a commit that referenced this pull request Jul 21, 2022
…ariables_create_table

Generalize create schema & table statements
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.

2 participants