Skip to content

Conversation

@shrshi
Copy link
Contributor

@shrshi shrshi commented Jan 16, 2026

Description

This PR reduces the number of configs for high-multiplicity join benchmarks from 2520 to 432, and for the sort-merge inner join benchmark from 1344 to 192.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@shrshi shrshi requested a review from a team as a code owner January 16, 2026 21:16
@shrshi shrshi requested review from bdice and karthikeyann January 16, 2026 21:16
@github-actions github-actions bot added the libcudf Affects libcudf (C++/CUDA) code. label Jan 16, 2026
@shrshi shrshi added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jan 16, 2026
.add_int64_axis("left_size", JOIN_SIZE_RANGE)
.add_int64_axis("right_size", JOIN_SIZE_RANGE)
.add_int64_axis("multiplicity", {10, 20, 50, 100, 1'000, 10'000, 50'000});
.add_int64_axis("multiplicity", {100, 1'000, 10'000, 50'000});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
.add_int64_axis("multiplicity", {100, 1'000, 10'000, 50'000});
.add_int64_axis("multiplicity", {100, 1'000, 10'000});

no big difference between 10'000 and 50'000 IMO. could be nonsense though

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement / enhancement to an existing function libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants