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

Documentation: union_relations uses "union all" #759

Closed
owenprough-sift opened this issue Jan 13, 2023 · 0 comments · Fixed by #760
Closed

Documentation: union_relations uses "union all" #759

owenprough-sift opened this issue Jan 13, 2023 · 0 comments · Fixed by #760
Labels
enhancement New feature or request triage

Comments

@owenprough-sift
Copy link
Contributor

Describe the feature

The documentation for union_relations should call out that the generated SQL uses union all. By the macro name, folks might assume it uses union (which is semantically different than union all).

Describe alternatives you've considered

N/A

Additional context

N/A

Who will this benefit?

Folks trying to understand the capabilities and limitations of this package's macros

Are you interested in contributing this feature?

Sure!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request triage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant