Skip to content

Commit d4f0c5f

Browse files
Update freshness config (#23)
* Update freshness config * snowflake and pr link * replace dbt 1.9.5 with 1.9.6 * source package ref
1 parent 4240b3c commit d4f0c5f

8 files changed

Lines changed: 67 additions & 33 deletions

File tree

Lines changed: 29 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,29 +1,35 @@
1-
## PR Overview
2-
**This PR will address the following Issue/Feature:**
1+
<!--
2+
Pre-Submission Reminders
3+
Before marking this PR as "ready for review":
34
4-
**This PR will result in the following new package version:**
5-
<!--- Please add details around your decision for breaking vs non-breaking version upgrade. If this is a breaking change, were backwards-compatible options explored? -->
5+
- `dbt run --full-refresh && dbt test`
6+
- `dbt run` && `dbt test` (if incremental models are present)
7+
- The related issue is linked, tagged, and appropriately assigned
8+
- Documentation and version updates are included, if applicable
9+
- `docs` have been regenerated (unless there are no code or YAML changes)
10+
- BuildKite integration tests are passing
11+
-->
612

7-
**Please provide the finalized CHANGELOG entry which details the relevant changes included in this PR:**
8-
<!--- Copy/paste the CHANGELOG for this version below. -->
13+
## PR Overview
14+
**Package version introduced in this PR:**
15+
16+
**This PR addresses the following Issue/Feature(s):**
17+
<!-- Add Issue # or internal ticket reference -->
918

10-
## PR Checklist
11-
### Basic Validation
12-
Please acknowledge that you have successfully performed the following commands locally:
13-
- [ ] dbt run –full-refresh && dbt test
14-
- [ ] dbt run (if incremental models are present) && dbt test
19+
**Summary of changes:**
20+
<!-- 1-2 sentences describing PR changes. -->
1521

16-
Before marking this PR as "ready for review" the following have been applied:
17-
- [ ] The appropriate issue has been linked, tagged, and properly assigned
18-
- [ ] All necessary documentation and version upgrades have been applied
19-
- [ ] docs were regenerated (unless this PR does not include any code or yml updates)
20-
- [ ] BuildKite integration tests are passing
21-
- [ ] Detailed validation steps have been provided below
2222

23-
### Detailed Validation
24-
Please share any and all of your validation steps:
25-
<!--- Provide the steps you took to validate your changes below. -->
23+
### Submission Checklist
24+
- [ ] Alignment meeting with the reviewer (if needed)
25+
- [ ] Timeline and validation requirements discussed
26+
- [ ] Provide validation details:
27+
- [ ] **Validation Steps:** Check for unintentional effects (e.g., add/run consistency & integrity tests)
28+
- [ ] **Testing Instructions:** Confirm the change addresses the issue(s)
29+
- [ ] **Focus Areas:** Complex logic or queries that need extra attention
2630

27-
### If you had to summarize this PR in an emoji, which would it be?
28-
<!--- For a complete list of markdown compatible emojis check our this git repo (https://gist.github.com/rxaviers/7360908) -->
29-
:dancer:
31+
### Changelog
32+
<!-- Recommend drafting changelog notes, then refining via ChatGPT using:
33+
"Draft a changelog entry based on the following notes." -->
34+
- [ ] Draft changelog for PR
35+
- [ ] Final changelog for release review

CHANGELOG.md

Lines changed: 27 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,34 @@
1-
# dbt_amazon_ads version.version
1+
# dbt_amazon_ads v0.5.0
2+
3+
[PR #23](https://github.com/fivetran/dbt_amazon_ads/pull/23) includes the following updates:
4+
5+
## Breaking Change for dbt Core < 1.9.6
6+
> *Note: This is not relevant to Fivetran Quickstart users.*
7+
8+
Migrated `freshness` from a top-level source property to a source `config` in alignment with [recent updates](https://github.com/dbt-labs/dbt-core/issues/11506) from dbt Core ([Source PR #24](https://github.com/fivetran/dbt_amazon_ads_source/pull/24)). This will resolve the following deprecation warning that users running dbt >= 1.9.6 may have received:
9+
10+
```
11+
[WARNING]: Deprecated functionality
12+
Found `freshness` as a top-level property of `amazon_ads` in file
13+
`models/src_amazon_ads.yml`. The `freshness` top-level property should be moved
14+
into the `config` of `amazon_ads`.
15+
```
16+
17+
**IMPORTANT:** Users running dbt Core < 1.9.6 will not be able to utilize freshness tests in this release or any subsequent releases, as older versions of dbt will not recognize freshness as a source `config` and therefore not run the tests.
18+
19+
If you are using dbt Core < 1.9.6 and want to continue running Amazon Ads freshness tests, please elect **one** of the following options:
20+
1. (Recommended) Upgrade to dbt Core >= 1.9.6
21+
2. Do not upgrade your installed version of the `amazon_ads` package. Pin your dependency on v0.4.0 in your `packages.yml` file.
22+
3. Utilize a dbt [override](https://docs.getdbt.com/reference/resource-properties/overrides) to overwrite the package's `amazon_ads` source and apply freshness via the [old](https://github.com/fivetran/dbt_amazon_ads_source/blob/main/models/src_amazon_ads.yml#L11-L13) top-level property route. This will require you to copy and paste the entirety of the `src_amazon_ads.yml` [file](https://github.com/fivetran/dbt_amazon_ads_source/blob/main/models/src_amazon_ads.yml#L4-L369) and add an `overrides: amazon_ads_source` property.
23+
24+
## Under the Hood
25+
- Updated the package maintainer PR template.
226

327
## Documentation
428
- Added Quickstart model counts to README. ([#21](https://github.com/fivetran/dbt_amazon_ads/pull/21))
529
- Corrected references to connectors and connections in the README. ([#21](https://github.com/fivetran/dbt_amazon_ads/pull/21))
30+
- Updated LICENSE.
31+
- Updated README headers.
632

733
# dbt_amazon_ads v0.4.0
834
[PR #16](https://github.com/fivetran/dbt_amazon_ads/pull/16) includes the following updates:

LICENSE

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@
186186
same "printed page" as the copyright notice for easier
187187
identification within third-party archives.
188188

189-
Copyright [yyyy] [name of copyright owner]
189+
Copyright © 2025 Fivetran Inc.
190190

191191
Licensed under the Apache License, Version 2.0 (the "License");
192192
you may not use this file except in compliance with the License.

README.md

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
1-
<p align="center">
1+
# Amazon Ads dbt Package ([Docs](https://fivetran.github.io/dbt_amazon_ads/))
2+
3+
<p align="left">
24
<a alt="License"
35
href="https://github.com/fivetran/dbt_amazon_ads/blob/main/LICENSE">
46
<img src="https://img.shields.io/badge/License-Apache%202.0-blue.svg" /></a>
@@ -10,7 +12,6 @@
1012
<img src="https://img.shields.io/badge/Contributions-welcome-blueviolet" /></a>
1113
</p>
1214

13-
# Amazon Ads dbt Package ([Docs](https://fivetran.github.io/dbt_amazon_ads/))
1415
## What does this dbt package do?
1516
- Produces modeled tables that leverage Amazon Ads data from [Fivetran's connector](https://fivetran.com/docs/applications/amazon-ads) in the format described by [this ERD](https://fivetran.com/docs/applications/amazon-ads#schemainformation) and builds off the output of our [Amazon Ads source package](https://github.com/fivetran/dbt_amazon_ads_source).
1617
- Provides insight into your ad performance across the following grains:
@@ -57,7 +58,7 @@ Include the following amazon_ads package version in your `packages.yml` file _if
5758
```yaml
5859
packages:
5960
- package: fivetran/amazon_ads
60-
version: [">=0.4.0", "<0.5.0"] # we recommend using ranges to capture non-breaking changes automatically
61+
version: [">=0.5.0", "<0.6.0"] # we recommend using ranges to capture non-breaking changes automatically
6162
```
6263

6364
Do NOT include the `amazon_ads_source` package in this file. The transformation package itself has a dependency on it and will install the source package as well.
@@ -157,7 +158,7 @@ This dbt package is dependent on the following dbt packages. Be aware that these
157158
```yml
158159
packages:
159160
- package: fivetran/amazon_ads_source
160-
version: [">=0.4.0", "<0.5.0"]
161+
version: [">=0.5.0", "<0.6.0"]
161162
162163
- package: fivetran/fivetran_utils
163164
version: [">=0.4.0", "<0.5.0"]

dbt_project.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
name: 'amazon_ads'
2-
version: '0.4.0'
2+
version: '0.5.0'
33
config-version: 2
44
require-dbt-version: [">=1.3.0", "<2.0.0"]
55
models:

integration_tests/dbt_project.yml

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

33
name: 'amazon_ads_integration_tests'
4-
version: '0.4.0'
4+
version: '0.5.0'
55

66
profile: 'integration_tests'
77

integration_tests/requirements.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,4 +4,5 @@ dbt-redshift>=1.3.0,<2.0.0
44
dbt-postgres>=1.3.0,<2.0.0
55
dbt-spark>=1.3.0,<2.0.0
66
dbt-spark[PyHive]>=1.3.0,<2.0.0
7-
dbt-databricks>=1.7.0,<2.0.0
7+
dbt-databricks>=1.6.0,<2.0.0
8+
certifi==2025.1.31

packages.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
packages:
22
- package: fivetran/amazon_ads_source
3-
version: [">=0.4.0", "<0.5.0"]
3+
version: [">=0.5.0", "<0.6.0"]

0 commit comments

Comments
 (0)