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

Make repartition a no-op when divisions match: also support list divs #10394

Closed
epizut opened this issue Jul 3, 2023 · 0 comments · Fixed by #10395
Closed

Make repartition a no-op when divisions match: also support list divs #10394

epizut opened this issue Jul 3, 2023 · 0 comments · Fixed by #10395
Labels
bug Something is broken good first issue Clearly described and easy to accomplish. Good for beginners to the project.

Comments

@epizut
Copy link
Contributor

epizut commented Jul 3, 2023

Initial issue #9922 fixed by #9924, failed to bypass identical repartition when the input divisions parameter is a list.

import dask
import dask.dataframe

dd = dask.datasets.timeseries()
dd.repartition(list(dd.divisions)).dask

image

Of course, if divisions is a tuple then the no-op bypass works well.
@github-actions github-actions bot added the needs triage Needs a response from a contributor label Jul 3, 2023
@epizut epizut mentioned this issue Jul 3, 2023
Merged
@fjetter fjetter added bug Something is broken good first issue Clearly described and easy to accomplish. Good for beginners to the project. and removed needs triage Needs a response from a contributor labels Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something is broken good first issue Clearly described and easy to accomplish. Good for beginners to the project.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make repartition a no-op when list of divisions match epizut/dask
2 participants
@epizut @fjetter