apache / datafusion-ballista

Apache DataFusion Ballista Distributed Query Engine
https://datafusion.apache.org/ballista
Apache License 2.0
1.39k stars 181 forks source link

Fix shuffle writer test #998

Closed Jefffrey closed 2 months ago

Jefffrey commented 2 months ago

Which issue does this PR close?

Closes #.

Rationale for this change

Running test locally on clean checkout had failing test suite. Looks like values of 1 and 2 were hashing to same partition causing only 1 partition, where test expects 2. Changing 2 to 3 seems sufficient to ensure it is hashed to separate partition.

What changes are included in this PR?

Are there any user-facing changes?