fivetran / fivetran/dbt_fivetran_utils

[FEATURE] - Add `persist_pass_through_columns` -capability for non-mapping passthrough variables

Open
#104 0 comments 0 reactions 0 assignees View on GitHub
type:enhancement
Dominant language
Shell
Stars
33
Forks
22
PR merge metrics
No merged PRs in 30d

Description

Currently when a non-mapped passthrough/list-style variable is passed to `persist_pass_through_columns` macro, no error occurs but blank fields are generated. This macro should be updated to handle either list- or dict- style variables. The logic used in [`fill_pass_through_columns`](https://github.com/fivetran/dbt_fivetran_utils/blob/releases/v0.4.latest/macros/fill_pass_through_columns.sql) is an example of how this could be done.

This will allow this macro to be brought into older packages, that might not yet use this macro and therefore are expecting list-style, with less friction on the user's end as they will not have to change their variables.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.