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

[FEA] support for arrays in BroadcastNestedLoopJoinExec and CartesianProductExec #2976

Closed
jlowe opened this issue Jul 20, 2021 · 1 comment · Fixed by #2989
Closed

[FEA] support for arrays in BroadcastNestedLoopJoinExec and CartesianProductExec #2976

jlowe opened this issue Jul 20, 2021 · 1 comment · Fixed by #2989
Assignees
@mythrocks
Labels
feature request New feature or request task Work required that improves the product but is not user facing

Comments

@jlowe
Copy link
Member

jlowe commented Jul 20, 2021

#2749 reverted #2702 due to a bug that it inadvertently exposed. After the bug is fixed, we need to restore the functionality from #2702.
@jlowe jlowe added feature request New feature or request ? - Needs Triage Need team to review and classify labels Jul 20, 2021
@sameerz
Copy link
Collaborator

sameerz commented Jul 20, 2021

The bug fix is being tracked in #2723

@mythrocks mythrocks self-assigned this Jul 21, 2021
@mythrocks mythrocks mentioned this issue Jul 21, 2021
Merged
@mythrocks mythrocks added task Work required that improves the product but is not user facing and removed ? - Needs Triage Need team to review and classify labels Jul 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mythrocks mythrocks

Labels
feature request New feature or request task Work required that improves the product but is not user facing
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Re-enable Array support in Cartesian Joins, Broadcast Nested Loop Joins mythrocks/spark-rapids
3 participants
@jlowe @mythrocks @sameerz