Skip to content

Commit

Permalink
Fix test_merge_p2p_shuffle_reused_dataframe_with_different_parameters (
Browse files Browse the repository at this point in the history
  • Loading branch information
hendrikmakait authored Dec 19, 2023
1 parent 25a1bfe commit 4d41d32
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion distributed/shuffle/tests/test_merge.py
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,9 @@ async def test_merge_p2p_shuffle_reused_dataframe_with_different_parameters(c, s
out = (
ddf1.merge(ddf2, left_on="a", right_on="x", shuffle="p2p")
# Vary the number of output partitions for the shuffles of dd2
.repartition(20).merge(ddf2, left_on="b", right_on="x", shuffle="p2p")
.repartition(npartitions=20).merge(
ddf2, left_on="b", right_on="x", shuffle="p2p"
)
)
# Generate unique shuffle IDs if the input frame is the same but
# parameters differ. Reusing shuffles in merges is dangerous because of the
Expand Down

0 comments on commit 4d41d32

Please sign in to comment.