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
[C++] Remove RelationInfo #35004
Comments
rtpsw
added a commit
to rtpsw/arrow
that referenced
this issue
Apr 10, 2023
icexelloss
changed the title
[C++] Remove output mapping translation
[C++] Remove Relation Info
Apr 10, 2023
If I recall correctly, earlier this was necessary because the as-of join implementation deduplicates columns between the join inputs, while Substrait generally expects the output to just be a concatenation of the columns. Do you still think you can remove this while handling as-of join correctly? |
cc @westonpace |
rtpsw
added a commit
to rtpsw/arrow
that referenced
this issue
Apr 14, 2023
icexelloss
pushed a commit
that referenced
this issue
Apr 20, 2023
liujiacheng777
pushed a commit
to LoongArch-Python/arrow
that referenced
this issue
May 11, 2023
See apache#35004 * Closes: apache#35004 Lead-authored-by: Yaron Gvili <rtpsw@hotmail.com> Co-authored-by: rtpsw <rtpsw@hotmail.com> Signed-off-by: Li Jin <ice.xelloss@gmail.com>
ArgusLi
pushed a commit
to Bit-Quill/arrow
that referenced
this issue
May 15, 2023
See apache#35004 * Closes: apache#35004 Lead-authored-by: Yaron Gvili <rtpsw@hotmail.com> Co-authored-by: rtpsw <rtpsw@hotmail.com> Signed-off-by: Li Jin <ice.xelloss@gmail.com>
rtpsw
added a commit
to rtpsw/arrow
that referenced
this issue
May 16, 2023
See apache#35004 * Closes: apache#35004 Lead-authored-by: Yaron Gvili <rtpsw@hotmail.com> Co-authored-by: rtpsw <rtpsw@hotmail.com> Signed-off-by: Li Jin <ice.xelloss@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the enhancement requested
Currently, there is a
RelationInfo
class in Acero whose purpose beyond that ofDeclarationInfo
is to specify the translation of an output mapping from input indices to output indices. However, Substrait column selection and emit documentation says the output mapping is already specified in terms of output indices.Component(s)
C++
The text was updated successfully, but these errors were encountered: