fivetran / fivetran/dbt_fivetran_log
[Feature] Update Data Models to Coincide with Connector Changes
- Dominant language
- No language data
- Stars
- 32
- Forks
- 30
- PR merge metrics
- No merged PRs in 30d
Description
### Is there an existing feature request for this?
- [X] I have searched the existing issues
### Describe the Feature
The Fivetran Platform connector will soon be seeing a few updates which will result in breaking changes for new connectors. The breaking changes include:
- The `connector_id` field in the **incremental_mar** table will be renamed to `connector_name`.
- We currently rename this field already in the [stg_fivetran_platform__incremental_mar model](https://github.com/fivetran/dbt_fivetran_log/blob/5fef3647a47e682236713c274895905ec5a81643/models/staging/stg_fivetran_platform__incremental_mar.sql#L10). Therefore, we will need to update this to ensure backwards and future compatibility following this connector update.
- The **connector** table will be renamed to **connection**.
- Additionally, the **connection** table PK will change `id` (previously `connector_id`) along with renaming changes to FK (will now be `connection_id`) references. The other tables with FK renaming which this data model leverages include:
- incremental_mar
- log
### Describe alternatives you've considered
_No response_
### Are you interested in contributing this feature?
- [ ] Yes.
- [ ] Yes, but I will need assistance and will schedule time during your [office hours](https://calendly.com/fivetran-solutions-team/fivetran-solutions-team-office-hours) for guidance.
- [ ] No.
### Anything else?
For existing connectors, this won't technically be breaking as both the old and new versions of the fields/tables will be generated. However, we will want to make sure the data model is updated to reflect the new changes while also being backwards compatible.
Please reference the internal ticket for additional details around these upcoming changes.
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.