fivetran / dbt_klaviyo

https://fivetran.github.io/dbt_klaviyo/
Apache License 2.0
5 stars 6 forks source link

Feature/unioned data #12

Closed fivetran-jamie closed 3 years ago

fivetran-jamie commented 3 years ago

Are you a current Fivetran customer?

no but this came from @pawelngei 's original PR https://github.com/fivetran/dbt_klaviyo/pull/11

What change(s) does this PR introduce?

incorporates the source_relation column produced by unioning-functionality in the source package. depends on https://github.com/fivetran/dbt_klaviyo_source/pull/9

note that i added the source_relation as a partition clause in the various window functions we use to attribute events

Does this PR introduce a breaking change?

Is this PR in response to a previously created Issue

How did you test the PR changes?

- [x] CircleCi - [ ] Other (please provide additional testing details below) **Select which warehouse(s) were used to test the PR** - [x] BigQuery - [x] Redshift - [x] Snowflake - [x] Postgres - [x] Databricks - [ ] Other (provide details below) **Provide an emoji that best describes your current mood** 📺 **Feedback** We are so excited you decided to contribute to the Fivetran community dbt package! We continue to work to improve the packages and would greatly appreciate your [feedback](https://www.surveymonkey.com/r/DQ7K7WW) on our existing dbt packages or what you'd like to see next.