Skip to content

fix: add new plugin#370

Open
sahanajadav wants to merge 3 commits into
mainfrom
feature/add_new_plugin
Open

fix: add new plugin#370
sahanajadav wants to merge 3 commits into
mainfrom
feature/add_new_plugin

Conversation

@sahanajadav
Copy link
Copy Markdown
Collaborator

@sahanajadav sahanajadav commented May 19, 2026

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

Add new plugins

  • analysis-icu
  • analysis-kuromoji

Related Tickets & Documents

  • Related Issue #
  • Closes #

QA Instructions, Screenshots, Recordings

  1. Verified plugin installation using:
curl -k -u admin:admin https://localhost:9200/_cat/plugins?v

Confirmed presence of:

  • analysis-icu
  • analysis-kuromoji

on all OpenSearch nodes.

  1. Functional validation for analysis-kuromoji plugin:

Created analyzer/index using kuromoji_tokenizer:

curl -k -u admin:admin -X PUT "https://localhost:9200/ja_test" \
-H "Content-Type: application/json" \
-d '{
  "settings": {
    "analysis": {
      "analyzer": {
        "ja_analyzer": {
          "type": "custom",
          "tokenizer": "kuromoji_tokenizer"
        }
      }
    }
  }
}'

Verified Japanese text tokenization using OpenSearch _analyze API:

curl -k -u admin:admin -X GET "https://localhost:9200/ja_test/_analyze" \
-H "Content-Type: application/json" \
-d '{
  "analyzer": "ja_analyzer",
  "text": "寿司が食べたい"
}'

This confirms the plugins are properly installed, loaded, and functioning at runtime.

Breaking Change checklist

If your PR includes any deployment or processing changes, please utilize this checklist:

  • Does it change any deployment parameters, logic of their working or rename them?
  • Did update from previous version tested with the same set of deployment parameters?

Added/updated tests?

  • Yes
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

[optional] Are there any things to highlight or double check?

[optional] What gif best describes this PR or how it makes you feel?

@github-actions
Copy link
Copy Markdown


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@github-actions github-actions Bot added the bug Something isn't working label May 19, 2026
Comment thread docker-opensearch/3x/docker/Dockerfile Outdated
@defesteban
Copy link
Copy Markdown
Collaborator

Please, add description how you tested new plugins.

@sahanajadav sahanajadav changed the title fix: add new plugin daft : fix: add new plugin May 19, 2026
@sahanajadav sahanajadav changed the title daft : fix: add new plugin fix: add new plugin May 19, 2026
@sahanajadav
Copy link
Copy Markdown
Collaborator Author

Please, add description how you tested new plugins.

Updated testing details.

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

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants