Skip to content
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

Remove link to customizing the name of a model's database identifier using the alias model config #6610

Merged
merged 2 commits into from
Dec 9, 2024

Conversation

dbeatty10
Copy link
Contributor

@dbeatty10 dbeatty10 commented Dec 6, 2024

What are you changing in this pull request and why?

There was a misunderstanding during implementation of #5460 that lead to a hyperlink that is about something totally different.

i.e., #5418 was referring to something like this:

select *
from {{ ref('stg_orders') }}  -- Look Ma, no relation alias!
left join {{ ref('stg_candies')}}  -- Look Pa, no relation alias here either!
    on stg_orders.candy_id = stg_candies.candy_id  -- Oopsie, the join condition here IS using relation aliases 💥

But the alias dbt model config is something totally different!

The alias config is used to give the model's an identifier in the database that differs from the model's name in dbt.

Checklist


🚀 Deployment available! Here are the direct links to the updated files:

Copy link

vercel bot commented Dec 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs-getdbt-com ✅ Ready (Inspect) Visit Preview Dec 9, 2024 9:58am

@dbeatty10 dbeatty10 marked this pull request as ready for review December 6, 2024 18:13
@dbeatty10 dbeatty10 requested a review from a team as a code owner December 6, 2024 18:13
@github-actions github-actions bot added content Improvements or additions to content size: x-small This change will take under 3 hours to fix. labels Dec 6, 2024
@mirnawong1 mirnawong1 enabled auto-merge December 9, 2024 09:54
@mirnawong1 mirnawong1 merged commit 151a9c0 into current Dec 9, 2024
9 checks passed
@mirnawong1 mirnawong1 deleted the dbeatty10-patch-1 branch December 9, 2024 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Improvements or additions to content size: x-small This change will take under 3 hours to fix.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants