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

chore(docs): update subquery documentation with more information #10361

Merged
merged 6 commits into from
May 7, 2024

Conversation

sanderson
Copy link
Contributor

@sanderson sanderson commented May 2, 2024

Which issue does this PR close?

This doesn't close an issue, but adds more detailed subquery documentation.

Part of #7013

Rationale for this change

Provides more information about how to use subqueries and what types of subqueries can be used.

What changes are included in this PR?

It just updates the subquery documentation.

Are these changes tested?

All examples in the proposed changes have been tested.

Are there any user-facing changes?

Nothing in the product itself. Just documentation.

@alamb alamb added the documentation Improvements or additions to documentation label May 3, 2024
Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Thank you @sanderson -- this looks really nice 😍

I think there are a few minor changes that could make this more accurate but overall this is looking 🌶️ 👌

Very nice

docs/source/user-guide/sql/subqueries.md Outdated Show resolved Hide resolved
docs/source/user-guide/sql/subqueries.md Outdated Show resolved Hide resolved
docs/source/user-guide/sql/subqueries.md Outdated Show resolved Hide resolved
docs/source/user-guide/sql/subqueries.md Show resolved Hide resolved
docs/source/user-guide/sql/subqueries.md Outdated Show resolved Hide resolved
docs/source/user-guide/sql/subqueries.md Outdated Show resolved Hide resolved
docs/source/user-guide/sql/subqueries.md Outdated Show resolved Hide resolved
@github-actions github-actions bot removed the documentation Improvements or additions to documentation label May 6, 2024
@sanderson sanderson requested a review from alamb May 6, 2024 23:30
Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Awesome - thank you so much @sanderson 🙏

@alamb alamb added the documentation Improvements or additions to documentation label May 7, 2024
@alamb alamb merged commit 0681004 into apache:main May 7, 2024
4 checks passed
@sanderson sanderson deleted the docs/subqueries branch May 7, 2024 17:21
findepi pushed a commit to findepi/datafusion that referenced this pull request Jul 16, 2024
…che#10361)

* chore(docs): update subquery documentation with more details and examples

* chore(docs): ran prettier on content updates

* Apply suggestions from code review

Co-authored-by: Andrew Lamb <[email protected]>

* chore(docs): address PR feedback for subqueries

* chore(docs): updated subquery doc formatting and style

---------

Co-authored-by: Andrew Lamb <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants