Skip to content

perf: add SIMD acceleration for min/max long/double expressions - #19691

Draft
jtuglu1 wants to merge 1 commit into
apache:masterfrom
jtuglu1:simd-min-max-exprs
Draft

jtuglu1 wants to merge 1 commit into
apache:masterfrom
jtuglu1:simd-min-max-exprs

Conversation

@jtuglu1

@jtuglu1 jtuglu1 commented Jul 15, 2026 •

Copy link
Copy Markdown
Contributor

Description

Adds simd processors for MIN/MAX expressions over long/double pair combinations. TODO: add benchmark results.

Release note

Adds simd processors for MIN/MAX expressions over long/double pair combinations.


This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • a release note entry in the PR description.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

@jtuglu1 jtuglu1 added this to the 38.0.0 milestone Jul 15, 2026
@jtuglu1
jtuglu1 requested a review from clintropolis July 15, 2026 16:33
@jtuglu1
jtuglu1 force-pushed the simd-min-max-exprs branch from 111b2a6 to e48df0c Compare July 15, 2026 18:05
@kfaraz

kfaraz commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Removing from Druid 38 milestone since this PR is still in draft.

@kfaraz kfaraz modified the milestones: 38.0.0, 39.0.0 Aug 1, 2026
@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown

This pull request has been marked as stale due to 60 days of inactivity.
It will be closed in 4 weeks if no further activity occurs. If you think
that's incorrect or this pull request should instead be reviewed, please simply
write any comment. Even if closed, you can still revive the PR at any time or
discuss it on the dev@druid.apache.org list.
Thank you for your contributions.

@github-actions github-actions Bot added the stale label Oct 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants