Skip to content

Commit d3ff7da

Browse files
Merge pull request #17 from fivetran/bug/snowflake-group
Bug/snowflake group
2 parents b7ec409 + aa584f6 commit d3ff7da

File tree

6 files changed

+25
-7
lines changed

6 files changed

+25
-7
lines changed

.buildkite/pipeline.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ steps:
5858
commands: |
5959
bash .buildkite/scripts/run_models.sh redshift
6060
61-
- label: ":bricks: Run Tests - Databricks"
61+
- label: ":databricks: Run Tests - Databricks"
6262
key: "run_dbt_databricks"
6363
plugins:
6464
- docker#v3.13.0:

CHANGELOG.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,11 @@
1+
# dbt_pendo_source v0.3.1
2+
## Bug Fixes
3+
- Updated readme for workaround if the pendo_<default_source_table_name>_identifer is having trouble with Snowflake reserved words. ([#17](https://github.com/fivetran/dbt_pendo_source/pull/17))
4+
## Under the Hood
5+
- Small adjustments to whitespace control in `src_pendo.yml`. ([#17](https://github.com/fivetran/dbt_pendo_source/pull/17))
6+
## Contributors
7+
- @RichardThRivera ([#16](https://github.com/fivetran/dbt_pendo_source/pull/16))
8+
19
# dbt_pendo_source v0.3.0
210
[PR #13](https://github.com/fivetran/dbt_pendo_source/pull/13) includes the following breaking changes:
311
## 🚨 Breaking Changes 🚨:

README.md

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,9 +97,20 @@ If an individual source table has a different name than the package expects, add
9797
```yml
9898
vars:
9999
pendo_source:
100-
pendo_<default_source_table_name>_identifier: your_table_name
100+
pendo_<default_source_table_name>_identifier: "your_table_name"
101101
```
102+
#### 🚨 Snowflake Users
103+
You may need to provide the case-sensitive spelling of your source tables that are also Snowflake reserved words.
102104

105+
In this package, this would apply to the `GROUP` source. If you are receiving errors for this source, include the following in your `dbt_project.yml` file:
106+
107+
```yml
108+
vars:
109+
pendo_group_identifier: '"Group"' # as an example, must include this quoting pattern and your exact casing
110+
111+
quoting:
112+
identifier: true
113+
```
103114
</details>
104115

105116
## (Optional) Step 5: Orchestrate your models with Fivetran Transformations for dbt Core™

dbt_project.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
config-version: 2
22
name: 'pendo_source'
3-
version: '0.3.0'
3+
version: '0.3.1'
44
require-dbt-version: [">=1.3.0", "<2.0.0"]
55
models:
66
pendo_source:

integration_tests/dbt_project.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,8 @@
11
config-version: 2
22

33
name: 'pendo_source_integration_tests'
4-
version: '0.3.0'
4+
version: '0.3.1'
55

6-
require-dbt-version: ">=0.20.0"
76
profile: 'integration_tests'
87

98
dispatch:

models/src_pendo.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -198,8 +198,8 @@ sources:
198198

199199
- name: group
200200
identifier: >
201-
{% if target.type == 'snowflake' %}{{ var('pendo_group_identifier', '"GROUP"') }}
202-
{% else %}{{ var('pendo_group_identifier', 'group') }}{% endif %}
201+
{%- if target.type == 'snowflake' -%}{{ var('pendo_group_identifier', '"GROUP"') }}
202+
{%- else -%}{{ var('pendo_group_identifier', 'group') }}{%- endif -%}
203203
description: Table capturing the product areas.
204204
columns:
205205
- name: app_id

0 commit comments

Comments
 (0)