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] Audit ShuffleExchangeExec #236

Closed
kuhushukla opened this issue Jun 19, 2020 · 2 comments
Closed

[FEA] Audit ShuffleExchangeExec #236

kuhushukla opened this issue Jun 19, 2020 · 2 comments
Assignees
@kuhushukla
Labels
feature request New feature or request P0 Must have for release
Milestone
Jul 20 - Jul 31

Comments

@kuhushukla
Copy link
Collaborator

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

  • Ensure Spark and Rapids plugin version of the exec match functionality.

  • Verify Config specific to the operator match.

  • Verify API is consistent and fully translated.

  • Port relevant tests.
@kuhushukla kuhushukla added feature request New feature or request ? - Needs Triage Need team to review and classify labels Jun 19, 2020
@kuhushukla kuhushukla removed the ? - Needs Triage Need team to review and classify label Jul 2, 2020
@sameerz sameerz added this to the Jul 6 - Jul 17 milestone Jul 6, 2020
@sameerz sameerz removed this from the Jul 6 - Jul 17 milestone Jul 7, 2020
@kuhushukla kuhushukla self-assigned this Jul 17, 2020
@sameerz sameerz added the P0 Must have for release label Jul 22, 2020
@kuhushukla
Copy link
Collaborator Author

Configs and API seems to be match well.

We need unit tests for RangePartitioning and other partitioning schemes to exercise parts where number of candidates is large etc. Especially PartitioningSuite , see #424

@kuhushukla
Copy link
Collaborator Author

Still needs metric tests like all other execs.

@kuhushukla kuhushukla added this to the Jul 20 - Jul 31 milestone Jul 24, 2020
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this issue Nov 30, 2023
@nvauto
[submodule-sync] bot-submodule-sync-branch-22.06 to branch-22.06 [ski…
4eb4889 
…p ci] [bot] (NVIDIA#236)

* Update submodule cudf to 0cc29a0

Signed-off-by: spark-rapids automation <[email protected]>

* Update submodule cudf to 3e1a345

Signed-off-by: spark-rapids automation <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kuhushukla kuhushukla

Labels
feature request New feature or request P0 Must have for release
Projects
None yet
Milestone
Jul 20 - Jul 31
Development

No branches or pull requests
2 participants
@sameerz @kuhushukla