Skip to content

adding examples to key value processor data prepper #11232

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
Nov 10, 2025
Merged

adding examples to key value processor data prepper #11232

merged 4 commits into from
Nov 10, 2025

Conversation

@AntonEliatra
Copy link
Contributor

@AntonEliatra AntonEliatra commented Oct 14, 2025

Description

adding examples to key value processor data prepper

Version

all

Checklist

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

@github-actions
Copy link

github-actions bot commented Oct 14, 2025

Thank you for submitting your PR. The PR states are In progress (or Draft) -> Tech review -> Doc review -> Editorial review -> Merged.

Before you submit your PR for doc review, make sure the content is technically accurate. If you need help finding a tech reviewer, tag a maintainer.

When you're ready for doc review, tag the assignee of this PR. The doc reviewer may push edits to the PR directly or leave comments and editorial suggestions for you to address (let us know in a comment if you have a preference). The doc reviewer will arrange for an editorial review.

@AntonEliatra
Copy link
Contributor Author

AntonEliatra commented Oct 14, 2025

@dlvenable can you please review this PR?

@AntonEliatra
Copy link
Contributor Author

AntonEliatra commented Oct 14, 2025

@dlvenable Can you review this PR please

@kolchfa-aws kolchfa-aws added Tech review PR: Tech review in progress backport 3.2 backport 3.3 and removed backport 3.2 labels Oct 14, 2025
@AntonEliatra
Update key-value.md
c7a9135 
Signed-off-by: AntonEliatra <[email protected]>
@dlvenable
Copy link
Member

dlvenable commented Nov 3, 2025

@kkondaka will do the technical review for this one.

@kkondaka
Copy link
Contributor

kkondaka commented Nov 3, 2025

@AntonEliatra I assume you have run the DataPrepper with the inputs mentioned in the documentation and the actual output seen in the opensearch index is pasted here. Is my assumption correct?

@AntonEliatra
Copy link
Contributor Author

AntonEliatra commented Nov 4, 2025

@kkondaka yes, that is correct

@AntonEliatra
Update key-value.md
8891097 
Signed-off-by: AntonEliatra <[email protected]>
kolchfa-aws
kolchfa-aws reviewed Nov 10, 2025
View reviewed changes
Copy link
Collaborator

@kolchfa-aws kolchfa-aws left a comment

Choose a reason for hiding this comment

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

Thank you, @AntonEliatra!

@kolchfa-aws kolchfa-aws assigned natebower and unassigned dlvenable Nov 10, 2025
@kolchfa-aws kolchfa-aws added Editorial review PR: Editorial review in progress and removed Tech review PR: Tech review in progress labels Nov 10, 2025
natebower
natebower approved these changes Nov 10, 2025
View reviewed changes
Copy link
Collaborator

@natebower natebower left a comment

Choose a reason for hiding this comment

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

LGTM

@natebower natebower removed the Editorial review PR: Editorial review in progress label Nov 10, 2025
@natebower natebower merged commit 71f8f6c into opensearch-project:main Nov 10, 2025
7 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Nov 10, 2025
@github-actions @kolchfa-aws
adding examples to key value processor data prepper (#11232)
415d455 
* adding examples to key value processor data prepper

Signed-off-by: Anton Rubin <[email protected]>

* Update key-value.md

Signed-off-by: AntonEliatra <[email protected]>

* Update key-value.md

Signed-off-by: AntonEliatra <[email protected]>

* Apply suggestions from code review

Signed-off-by: kolchfa-aws <[email protected]>

---------

Signed-off-by: Anton Rubin <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
(cherry picked from commit 71f8f6c)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot opensearch-trigger-bot bot mentioned this pull request Nov 10, 2025
Merged
github-actions bot pushed a commit that referenced this pull request Nov 10, 2025
@opensearch-trigger-bot
adding examples to key value processor data prepper (#11232) (#11513)
7f1ae4c
aryasoni98 pushed a commit to aryasoni98/documentation-website that referenced this pull request Nov 11, 2025
@AntonEliatra @kolchfa-aws @aryasoni98
adding examples to key value processor data prepper (opensearch-proje…
421b438 
…ct#11232)

* adding examples to key value processor data prepper

Signed-off-by: Anton Rubin <[email protected]>

* Update key-value.md

Signed-off-by: AntonEliatra <[email protected]>

* Update key-value.md

Signed-off-by: AntonEliatra <[email protected]>

* Apply suggestions from code review

Signed-off-by: kolchfa-aws <[email protected]>

---------

Signed-off-by: Anton Rubin <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Arya Soni <[email protected]>
@AntonEliatra AntonEliatra deleted the adding-example-for-key-value-processor branch November 11, 2025 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kolchfa-aws kolchfa-aws kolchfa-aws left review comments

@natebower natebower natebower approved these changes

@AMoo-Miki AMoo-Miki Awaiting requested review from AMoo-Miki AMoo-Miki is a code owner

@dlvenable dlvenable Awaiting requested review from dlvenable dlvenable is a code owner

@epugh epugh Awaiting requested review from epugh epugh is a code owner

@sumobrian sumobrian Awaiting requested review from sumobrian sumobrian is a code owner

+1 more reviewer

@kkondaka kkondaka kkondaka approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@natebower natebower

@kolchfa-aws kolchfa-aws

Labels

backport 3.3

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@AntonEliatra @dlvenable @kkondaka @natebower @kolchfa-aws