Introducing indexing & deletion strategy planner interfaces by shank9918 · Pull Request #20585 · opensearch-project/OpenSearch

shank9918 · 2026-02-10T12:09:21Z

Description

Introduce Strategy Planner Interfaces

Refactor indexing and deletion planning logic into dedicated planner interfaces responsible for primary and non-primary execution. Default planner implementations would encapsulate the existing InternalEngine behavior, while enabling alternative engines to provide custom planners or reuse shared implementations.

Delegate Planning via Composition

Update Engine implementations to delegate planning decisions to injected planner instances. The default behavior for InternalEngine should remain unchanged, ensuring backward compatibility.

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

[Feature Request] Refactor Engine internals to decouple operation/result types and introduce pluggable planning strategies

Check List

Functionality includes testing.
API changes companion pull request created, if applicable.
Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

coderabbitai · 2026-02-10T12:09:33Z

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

server/src/main/java/org/opensearch/index/engine/Engine.java

github-actions · 2026-02-11T05:34:09Z

❌ Gradle check result for 944cf1a: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

github-actions · 2026-02-11T06:03:45Z

❌ Gradle check result for 479e74f: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

server/src/main/java/org/opensearch/index/engine/DeletionStrategy.java

github-actions · 2026-02-11T07:56:35Z

❌ Gradle check result for eecb2d0: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

github-actions · 2026-02-11T12:06:13Z

❌ Gradle check result for 6be6a79: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

github-actions · 2026-02-12T06:51:41Z

❌ Gradle check result for 83bcfc7: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

github-actions · 2026-02-12T08:49:25Z

❌ Gradle check result for 83bcfc7: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

github-actions · 2026-02-26T16:40:38Z

Persistent review updated to latest commit cf08e29

server/src/main/java/org/opensearch/index/engine/InternalEngine.java

github-actions · 2026-02-26T17:13:12Z

Persistent review updated to latest commit a681e1e

github-actions · 2026-02-26T17:31:37Z

Persistent review updated to latest commit 22e6d16

github-actions · 2026-02-26T17:45:31Z

❌ Gradle check result for 22e6d16: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

github-actions · 2026-02-27T07:44:56Z

Persistent review updated to latest commit 3c68659

github-actions · 2026-02-27T08:28:16Z

❌ Gradle check result for 3c68659: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

github-actions · 2026-02-27T09:18:58Z

Persistent review updated to latest commit dab203a

github-actions · 2026-02-27T11:20:27Z

❌ Gradle check result for dab203a: null

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Signed-off-by: Shashank Gowri <shnkgo@amazon.com>

github-actions · 2026-02-27T11:30:35Z

Persistent review updated to latest commit d85601b

github-actions · 2026-02-27T12:59:36Z

✅ Gradle check result for d85601b: SUCCESS

…ch-project#20585) Signed-off-by: Shashank Gowri <shnkgo@amazon.com>

…ch-project#20585) Signed-off-by: Shashank Gowri <shnkgo@amazon.com> Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com>

…#20729) * Implement FieldMappingIngestionMessageMapper for pull-based ingestion Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Add changelog Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Address bot comment Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Address comments Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Address comments Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Remove affiliation column for emeritus maintainers (#20725) Emeritus maintainers are not active in the project, therefore I don't see a lot of value in tracking their affiliation. Signed-off-by: Andrew Ross <andrross@amazon.com> Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Add bitmap64 query support (#20606) --------- Signed-off-by: Divya <DIVYA2@ibm.com> Signed-off-by: Divya <divyaruhil999@gmail.com> Co-authored-by: Divya <DIVYA2@ibm.com> Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * fix stream transport TLS cert hot-reload by using live SSLContext from SecureTransportSettingsProvider (#20734) Signed-off-by: Rishabh Maurya <rishabhmaurya05@gmail.com> Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Bump OpenTelemetry to 1.59.0 and OpenTelemetry Semconv to 1.40.0 (#20737) Signed-off-by: Andriy Redko <drreta@gmail.com> Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * [Pull-based Ingestion] Remove experimental tag for pull-based ingestion (#20704) * remove experimental tag for pull-based ingestion Signed-off-by: Varun Bharadwaj <varunbharadwaj1995@gmail.com> * update BroadcastRequest to be marked as public API Signed-off-by: Varun Bharadwaj <varunbharadwaj1995@gmail.com> --------- Signed-off-by: Varun Bharadwaj <varunbharadwaj1995@gmail.com> Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Bump Apache Lucene from 10.3.2 to 10.4.0 (#20735) Signed-off-by: Ankit Jain <jainankitk@apache.org> Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Minor Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Fix spotless check Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Address bot comment Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Make id mandatory when id field provided Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Fix spotless check Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Introducing indexing & deletion strategy planner interfaces (#20585) Signed-off-by: Shashank Gowri <shnkgo@amazon.com> Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Add changelog Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Refactor Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Fix spotless check Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Empty commit Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Remove duplicate changelog Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> * Empty commit Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> --------- Signed-off-by: Rishab Nahata <rishabnahata07@gmail.com> Signed-off-by: Andrew Ross <andrross@amazon.com> Signed-off-by: Divya <DIVYA2@ibm.com> Signed-off-by: Divya <divyaruhil999@gmail.com> Signed-off-by: Rishabh Maurya <rishabhmaurya05@gmail.com> Signed-off-by: Andriy Redko <drreta@gmail.com> Signed-off-by: Varun Bharadwaj <varunbharadwaj1995@gmail.com> Signed-off-by: Ankit Jain <jainankitk@apache.org> Signed-off-by: Shashank Gowri <shnkgo@amazon.com> Co-authored-by: Andrew Ross <andrross@amazon.com> Co-authored-by: Divya <117009486+divyaruhil@users.noreply.github.com> Co-authored-by: Divya <DIVYA2@ibm.com> Co-authored-by: Rishabh Maurya <rishabhmaurya05@gmail.com> Co-authored-by: Andriy Redko <drreta@gmail.com> Co-authored-by: Varun Bharadwaj <varunbharadwaj1995@gmail.com> Co-authored-by: Ankit Jain <jainankitk@apache.org> Co-authored-by: Shashank Gowri <shashankgowri@gmail.com>

shank9918 changed the title ~~Introducing indexing & deletion strategy planner interfaces and refac…~~ Decouple operation/result types and introduce pluggable planning strategies Feb 10, 2026

mgodwan reviewed Feb 10, 2026

View reviewed changes

server/src/main/java/org/opensearch/index/engine/Engine.java Outdated Show resolved Hide resolved

shank9918 force-pushed the refactored-engine branch 3 times, most recently from 344f4b8 to 944cf1a Compare February 11, 2026 05:26

shank9918 force-pushed the refactored-engine branch 3 times, most recently from 9513ca5 to 479e74f Compare February 11, 2026 05:49

shank9918 force-pushed the refactored-engine branch from 479e74f to eecb2d0 Compare February 11, 2026 06:12

Bukhtawar reviewed Feb 11, 2026

View reviewed changes

server/src/main/java/org/opensearch/index/engine/DeletionStrategy.java Outdated Show resolved Hide resolved

shank9918 changed the title ~~Decouple operation/result types and introduce pluggable planning strategies~~ Introducing indexing & deletion strategy planner interfaces Feb 11, 2026

shank9918 force-pushed the refactored-engine branch from eecb2d0 to 6be6a79 Compare February 11, 2026 11:35

shank9918 force-pushed the refactored-engine branch 2 times, most recently from 3c3a9cd to 83bcfc7 Compare February 12, 2026 06:05

shank9918 closed this Feb 12, 2026

shank9918 reopened this Feb 12, 2026

shank9918 force-pushed the refactored-engine branch from 614ddcb to cf08e29 Compare February 26, 2026 16:39

mgodwan reviewed Feb 26, 2026

View reviewed changes

server/src/main/java/org/opensearch/index/engine/InternalEngine.java Show resolved Hide resolved

shank9918 force-pushed the refactored-engine branch from cf08e29 to a681e1e Compare February 26, 2026 17:11

Bukhtawar approved these changes Feb 26, 2026

View reviewed changes

shank9918 force-pushed the refactored-engine branch from a681e1e to 22e6d16 Compare February 26, 2026 17:29

shank9918 force-pushed the refactored-engine branch from 22e6d16 to 3c68659 Compare February 27, 2026 07:43

shank9918 force-pushed the refactored-engine branch from 3c68659 to dab203a Compare February 27, 2026 09:17

Introducing indexing & deletion strategy planner interfaces

d85601b

Signed-off-by: Shashank Gowri <shnkgo@amazon.com>

shank9918 force-pushed the refactored-engine branch from dab203a to d85601b Compare February 27, 2026 11:29

Bukhtawar approved these changes Feb 27, 2026

View reviewed changes

Bukhtawar merged commit 59be6ae into opensearch-project:main Feb 27, 2026
35 checks passed

opensearch-ci-bot mentioned this pull request Feb 27, 2026

[AUTOCUT] Gradle Check Flaky Test Report for AwarenessAllocationIT #17930

Open

imRishN pushed a commit to imRishN/OpenSearch that referenced this pull request Feb 28, 2026

Introducing indexing & deletion strategy planner interfaces (opensear…

67f4ea2

…ch-project#20585) Signed-off-by: Shashank Gowri <shnkgo@amazon.com>

This was referenced Mar 9, 2026

[AUTOCUT] Gradle Check Flaky Test Report for IndexShardTests #14321

Open

[AUTOCUT] Gradle Check Flaky Test Report for SegmentReplicationWithNodeToNodeIndexShardTests #20860

Open

This was referenced Mar 20, 2026

[BUG] Plugin is not compatible with the latest OpenSearch 3.6.0-SNAPSHOT opensearch-project/cross-cluster-replication#1646

Closed

CCR: Remove non-primary origin assertions from operation planners #20952

Closed

andrross mentioned this pull request Mar 21, 2026

Add release notes for 3.6.0 andrross/OpenSearch#216

Open

1 task

Conversation

shank9918 commented Feb 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Related Issues

Check List

Uh oh!

coderabbitai bot commented Feb 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

Uh oh!

github-actions bot commented Feb 11, 2026

Uh oh!

github-actions bot commented Feb 11, 2026

Uh oh!

Uh oh!

github-actions bot commented Feb 11, 2026

Uh oh!

github-actions bot commented Feb 11, 2026

Uh oh!

github-actions bot commented Feb 12, 2026

Uh oh!

github-actions bot commented Feb 12, 2026

Uh oh!

github-actions bot commented Feb 26, 2026

Uh oh!

Uh oh!

github-actions bot commented Feb 26, 2026

Uh oh!

github-actions bot commented Feb 26, 2026

Uh oh!

github-actions bot commented Feb 26, 2026

Uh oh!

github-actions bot commented Feb 27, 2026

Uh oh!

github-actions bot commented Feb 27, 2026

Uh oh!

github-actions bot commented Feb 27, 2026

Uh oh!

github-actions bot commented Feb 27, 2026

Uh oh!

github-actions bot commented Feb 27, 2026

Uh oh!

github-actions bot commented Feb 27, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

shank9918 commented Feb 10, 2026 •

edited

Loading

coderabbitai bot commented Feb 10, 2026 •

edited

Loading