Skip to content

perf: use direct selection for safe conditional joins - #1691

Open
samukweku wants to merge 3 commits into
devfrom
codex/issue-1687-matches
Open

perf: use direct selection for safe conditional joins#1691
samukweku wants to merge 3 commits into
devfrom
codex/issue-1687-matches

Conversation

@samukweku

Copy link
Copy Markdown
Collaborator

Summary

  • use janitor-rs direct first/last selection for safe range-join remainder predicates
  • gate the path to homogeneous, supported, non-null dtypes
  • retain the matches-tape fallback for mixed/nullable inputs, keep="all", unavailable exports, and other unsupported cases
  • add dispatch coverage

Verification

  • 279 conditional-join tests passed, 1 skipped
  • pre-commit hooks passed

Depends on pyjanitor-devs/janitor-rs#108.

Related issue: #1687

Please do not auto-close #1687; it will be closed manually after the related PRs are merged.

@github-actions

github-actions Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://pyjanitor-devs.github.io/pyjanitor/pr-preview/pr-1691/

Built to branch gh-pages at 2026-08-26 07:52 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@samukweku samukweku self-assigned this Aug 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[PERF] Reduce or eliminate intermediate matches arrays in conditional_join

1 participant