fivetran / dbt_ad_reporting

Fivetran's ad reporting dbt package. Combine your Facebook, Google, Pinterest, LinkedIn, Twitter, Snapchat, Microsoft, TikTok, Reddit, Amazon, and Apple Search advertising metrics using this package.
https://fivetran.github.io/dbt_ad_reporting/#!/overview
Apache License 2.0
142 stars 56 forks source link

[Bug] Update semantic model name #104

Closed Jstein77 closed 10 months ago

Jstein77 commented 11 months ago

Is there an existing issue for this?

Describe the issue

We released an update to dbt-semantic-interfaces that reserves the dunder(__) character as a keyword, and does not allow dunders in semantic model names. The package currently has a semantic model named ad_reporting__ad_report. This is causing dbt parse jobs to fail on dbt v1.6.5.

To fix this bug we need to updated the name of the semantic models to remove dunders.

Relevant error log or model output

No response

Expected behavior

dbt parse should succeed when we remove dunders from the semantic model names.

dbt Project configurations

vars: ad_reporting: ad_reportingaccount_passthrough_metrics: [] ad_reporting__campaign_passthrough_metrics: [] ad_reportingad_group_passthrough_metrics: [] ad_reportingad_passthrough_metrics: [] ad_reporting__keyword_passthrough_metrics: [] ad_reportingsearch_passthrough_metrics: [] "dbt_date:time_zone": "America/Los_Angeles"

Package versions

packages:

What database are you using dbt with?

snowflake

dbt Version

1.6.5

Additional Context

No response

Are you willing to open a PR to help address this issue?

raphaelvarieras commented 11 months ago

I see this is currently a p4 ticket - does that mean you recommend I disable the package and disable all my dependencies on it for now?

fivetran-joemarkiewicz commented 10 months ago

I apologize, but I didn't seem to close this after merging and releasing the update. This was live in the v1.6.1 release of the Ad Reporting package.

Closing out this issue and thank you @Jstein77 for your help in opening the PR and addressing the issue.