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

Update rcte plugin to allow the user to specify union_all in rcte #2107

Merged
merged 1 commit into from
Dec 19, 2023

Conversation

JonathanFrias
Copy link
Contributor

Support UNION style recursive common table expressions to remove duplicates by exposing a passthrough option :union_all

Support UNION style recursive common table expressions to remove
duplicates by exposing a passthrough option :union_all
@JonathanFrias JonathanFrias changed the title Update rcte plugin to allow the user to specify union_all of rcte Update rcte plugin to allow the user to specify union_all in rcte Dec 19, 2023
@jeremyevans
Copy link
Owner

Can you explain why you want this? I'm not opposed to this, but I'm not sure in which cases it would be helpful. It doesn't seem like it would help lazy loading, since that should not result in duplicate rows in a tree. Eager loading where the base results are in different levels of the tree maybe?

@JonathanFrias
Copy link
Contributor Author

I have a tree based data structure where there could be many duplicates in many levels in the tree, but for my use case I only care if there's 1, so I'm looking at the feasibility of using this plugin.

@jeremyevans jeremyevans merged commit 0167ddf into jeremyevans:master Dec 19, 2023
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants