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

Tweak blog landing page #6930

Merged
merged 4 commits into from
Feb 19, 2025
Merged

Tweak blog landing page #6930

merged 4 commits into from
Feb 19, 2025

Conversation

joellabes
Copy link
Contributor

What are you changing in this pull request and why?

  • Hide the header bar on the blog
    image
  • Remove the callout at the top of the sql comprehension article, leaving just the bottom one to keep people going once they're primed.

🚨 Depends on #6929 - don't merge until then!

@joellabes joellabes requested a review from a team as a code owner February 19, 2025 18:46
Copy link

vercel bot commented Feb 19, 2025

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

Name Status Preview Updated (UTC)
docs-getdbt-com ✅ Ready (Inspect) Visit Preview Feb 19, 2025 8:01pm

@github-actions github-actions bot added content Improvements or additions to content developer blog This content fits on the developer blog. labels Feb 19, 2025
@joellabes joellabes requested a review from jasnonaz February 19, 2025 18:46
@github-actions github-actions bot added the size: medium This change will take up to a week to address label Feb 19, 2025

## What if we wanted it to be faster?

Since running the code is slow, one way to get results faster is to not run the code. Since v1.0, dbt's parser has [used a static analyzer](https://github.com/dbt-labs/dbt-core/blob/main/docs/guides/parsing-vs-compilation-vs-runtime.md#:~:text=Simple%20Jinja%2DSQL%20models%20(using%20just%20ref()%2C%20source()%2C%20%26/or%20config()%20with%20literal%20inputs)%20are%20also%20statically%20analyzed%2C%20using%20a%20thing%20we%20built.%20This%20is%20very%20fast%20(~0.3%20ms)) to resolve refs when possible, which is [about 3x faster](https://docs.getdbt.com/reference/parsing#:~:text=For%20now%2C%20the%20static%20parser,speedup%20in%20the%20model%20parser) than going through the whole rigmarole above.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [vale] reported by reviewdog 🐶
[custom.Typos] Oops there's a typo -- did you really mean 'v1.0'?


<Lightbox src="/img/blog/2025-02-19-faster-project-parsing-with-rust/evaluation_strategies_2.png" width="100%" />

Our analysis in the leadup to dbt v1.0 showed that the static analyzer could handle 60% of models. Evaluating refs 30x faster in 60% of models would itself be great.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [vale] reported by reviewdog 🐶
[custom.Typos] Oops there's a typo -- did you really mean 'v1.0'?

- If your project isn't currently eligible for partial parsing, cold parses in Rust are fast enough to make it a moot point.
- Regardless of how your project parses today, your project will feel like it's a couple of orders of magnitude smaller than it is.

## We're just getting started
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [vale] reported by reviewdog 🐶
[custom.SentenceCaseHeaders] 'We're just getting started' should use sentence-style capitalization. Try 'We re just getting started' instead.

@github-actions github-actions bot added size: x-small This change will take under 3 hours to fix. and removed size: medium This change will take up to a week to address labels Feb 19, 2025
@joellabes joellabes merged commit 475f3f7 into current Feb 19, 2025
7 checks passed
@joellabes joellabes deleted the tweak-blog-landing-page branch February 19, 2025 20:01
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 developer blog This content fits on the developer blog. 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