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

Improve GpuRangePartitioning implementation #1355

Closed
andygrove opened this issue Dec 9, 2020 · 1 comment
Closed

Improve GpuRangePartitioning implementation #1355

andygrove opened this issue Dec 9, 2020 · 1 comment
Assignees
@gerashegalov @kuhushukla
Labels
P2 Not required for release task Work required that improves the product but is not user facing

Comments

@andygrove
Copy link
Contributor

Is your feature request related to a problem? Please describe.

As mentioned in the conversation on #1350 there is no point in passing a GpuRangePartitioner into GpuRangePartitioning and we should clean this code up, Also, GpuRangePartitioning has an unused rangeBounds variable that can be removed.
@andygrove andygrove added feature request New feature or request ? - Needs Triage Need team to review and classify labels Dec 9, 2020
@andygrove andygrove mentioned this issue Dec 9, 2020
Merged
@kuhushukla kuhushukla self-assigned this Dec 9, 2020
@sameerz sameerz added task Work required that improves the product but is not user facing and removed ? - Needs Triage Need team to review and classify feature request New feature or request labels Dec 15, 2020
@sameerz sameerz changed the title [FEA] Improve GpuRangePartitioning implementation Improve GpuRangePartitioning implementation Dec 15, 2020
@kuhushukla kuhushukla added the P2 Not required for release label Jan 6, 2021
@sameerz
Copy link
Collaborator

sameerz commented Mar 2, 2021

Completed by @revans2

@sameerz sameerz closed this as completed Mar 2, 2021
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this issue Nov 30, 2023
@nvauto
Update submodule cudf to 28b5b6e59e49343496e441ea45a843b4dd8d1bf3 (NV…
9a9cbfb 
…IDIA#1355)

Signed-off-by: spark-rapids automation <70000568+nvauto@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gerashegalov gerashegalov

@kuhushukla kuhushukla

Labels
P2 Not required for release task Work required that improves the product but is not user facing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@andygrove @gerashegalov @sameerz @kuhushukla