-
Notifications
You must be signed in to change notification settings - Fork 2
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
update is_most_recent_record in sub history #15
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fivetran-reneeli Code changes look good! Suggestion for the changelog, and also left a comment in Height re the validation. Thanks!
CHANGELOG.md
Outdated
# dbt_recharge_source v0.3.1 | ||
[PR #15](https://github.com/fivetran/dbt_recharge_source/pull/15) includes the following updates: | ||
## Bugfix | ||
- Updates the `is_most_recent_record` field in `stg_recharge__subscription_history` to partition on `id`, or `subscription_id` if the former is not available in the source table. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would add an explanation of why this had to be done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome--lgtm!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @fivetran-reneeli for this quick fix. These changes look good to me!
PR Overview
This PR will address the following Issue/Feature: Issue #14
internal ticket where a customer was seeing a partition by constant expression issue due to a null value as the partition key resulting from having
id
in their source subscription_history table instead ofsubscription_id
. To fix this we separated out the partition so that the coalesce betweenid
andsubscription_id
occurred first.This PR will result in the following new package version:
v0.3.1
Please provide the finalized CHANGELOG entry which details the relevant changes included in this PR:
Bugfix
is_most_recent_record
field instg_recharge__subscription_history
to partition onid
, orsubscription_id
if the former is not available in the source table.PR Checklist
Basic Validation
Please acknowledge that you have successfully performed the following commands locally:
Before marking this PR as "ready for review" the following have been applied:
Detailed Validation
Please share any and all of your validation steps:
Hex notebook linked in ticket!
If you had to summarize this PR in an emoji, which would it be?
💃