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

Link directly to section of Expressions doc from Aggregate Functions page #5166

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

philrz
Copy link
Contributor

@philrz philrz commented Jul 3, 2024

In a recent community Slack thread, a user was learning for the first time about how aggregate functions behave differently when called in "expression context" vs. "summarization context". When the user asked if this was documented anywhere, I checked and confirmed that it was, but looking at it through a user's eyes I recognized that the link on this page to the top of Expressions page was probably not ideal since it's such a big page. Since there's a section in Expressions that speaks specifically to difference vs. summarization, here I'm proposing we link to that directly instead. If the user gets what they need from that section and then want to stick around and learn about all things Expression from the rest of the page, great!

@philrz philrz requested a review from a team July 3, 2024 17:49
@philrz philrz self-assigned this Jul 3, 2024
@philrz philrz merged commit a3b40b2 into main Jul 8, 2024
4 checks passed
@philrz philrz deleted the docs-better-agg-expr-link branch July 8, 2024 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants