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] Improve coalesce performance #26

Closed
revans2 opened this issue May 28, 2020 · 1 comment
Closed

[FEA] Improve coalesce performance #26

revans2 opened this issue May 28, 2020 · 1 comment
Labels
cudf_dependency An issue or PR with this label depends on a new feature in cudf feature request New feature or request P2 Not required for release performance A performance related task/issue SQL part of the SQL/Dataframe plugin wontfix This will not be worked on

Comments

@revans2
Copy link
Collaborator

revans2 commented May 28, 2020

Is your feature request related to a problem? Please describe.
The current coalesce operation chains isNull and ifElse operations to make it work.

rapidsai/cudf#3754 was filed for a cudf implementation that makes this much simpler/more efficient.
@revans2 revans2 added feature request New feature or request ? - Needs Triage Need team to review and classify SQL part of the SQL/Dataframe plugin performance A performance related task/issue labels May 28, 2020
@sameerz sameerz added P2 Not required for release and removed ? - Needs Triage Need team to review and classify labels Oct 13, 2020
wjxiz1992 added a commit to wjxiz1992/spark-rapids that referenced this issue Oct 29, 2020
@wjxiz1992 @anfeng
Update hyperlinks in README.md (NVIDIA#26)
c04220f
@revans2 revans2 added the cudf_dependency An issue or PR with this label depends on a new feature in cudf label Feb 18, 2021
@zhanga5 zhanga5 mentioned this issue Aug 12, 2021
Merged
@mattahrens
Copy link
Collaborator

Closing as won't fix for now

@sameerz sameerz added the wontfix This will not be worked on label May 26, 2022
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this issue Nov 30, 2023
@nvauto
Update submodule cudf to 5257f34 (NVIDIA#26)
e03e6dd 
Signed-off-by: spark-rapids automation <70000568+nvauto@users.noreply.github.com>
sperlingxx pushed a commit to sperlingxx/spark-rapids that referenced this issue Jan 19, 2024
@firestarman
Revert "Fix the canonicalizing for GPU file scan (NVIDIA#10137)" (NVI…
c17eb8a 
…DIA#26)

This reverts commit ace4870.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cudf_dependency An issue or PR with this label depends on a new feature in cudf feature request New feature or request P2 Not required for release performance A performance related task/issue SQL part of the SQL/Dataframe plugin wontfix This will not be worked on
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@revans2 @mattahrens @sameerz