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.

Sign up for GitHub

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

Jump to bottom

ReferenceRel to capture DAGs/Spool semantics #283

Open
curino opened this issue Aug 9, 2022 · 1 comment
Open

ReferenceRel to capture DAGs/Spool semantics #283

curino opened this issue Aug 9, 2022 · 1 comment
Labels
awaiting-user-input This issue is waiting on further input from users

Comments

@curino
Copy link
Contributor

curino commented Aug 9, 2022

Given that Plan already support multiple Rel (repeated Rel relations), it is very valuable to capture overlaps among them. This is valuable to reduce redundancy, as well as support multi-output (in combination with #128).
curino added a commit to curino/substrait that referenced this issue Aug 9, 2022
@curino
feat: introducing a ReferenceRel operator to express DAG/Spool. (subs…
0dcbe8a 
…trait-io#283)
curino added a commit to curino/substrait that referenced this issue Aug 9, 2022
@curino
feat: removing spurious change. (substrait-io#283)
e041822
curino added a commit to curino/substrait that referenced this issue Aug 9, 2022
@curino
feat: improving docs (substrait-io#283)
056073d
curino added a commit to curino/substrait that referenced this issue Aug 10, 2022
@curino
feat: introducing a ReferenceRel operator to express DAG/Spool. (subs…
6272d93 
…trait-io#283)
curino added a commit to curino/substrait that referenced this issue Aug 10, 2022
@curino
feat: improving docs (substrait-io#283)
225846c
jacques-n added a commit that referenced this issue Aug 11, 2022
@curino @jacques-n
Implementation for ReferenceRel (#284)
70736ee 
* feat: introducing a ReferenceRel operator to express DAG/Spool. (#283)

Co-authored-by: Jacques Nadeau <[email protected]>
richtia pushed a commit to richtia/substrait that referenced this issue Aug 16, 2022
@curino @jacques-n @richtia
feat: introducing a ReferenceRel operator to express DAG/Spool (subst…
cad64c1 
…rait-io#283)

Co-authored-by: Jacques Nadeau <[email protected]>
@westonpace
Copy link
Member

@curino was this issue closed by #284 ?

@westonpace westonpace added the awaiting-user-input This issue is waiting on further input from users label Mar 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
awaiting-user-input This issue is waiting on further input from users
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@curino @westonpace