Skip to content
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.

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

Add rename ingest processor #6951

Merged
merged 7 commits into from
Apr 22, 2024
Merged

Conversation

gaobinlong
Copy link
Contributor

@gaobinlong gaobinlong commented Apr 15, 2024

Description

Add documentation for the existing rename ingest processor, this documentation needs to be backported to all versions.

Issues Resolved

Closes #6950

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.

Signed-off-by: Melissa Vagi <[email protected]>
@vagimeli vagimeli added 5 - Editorial review PR: Editorial review in progress and removed 4 - Doc review PR: Doc review in progress labels Apr 16, 2024
Signed-off-by: Melissa Vagi <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Copy link
Contributor

@vagimeli vagimeli left a comment

Choose a reason for hiding this comment

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

Doc review complete. Minimal edits.

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.

@gaobinlong @vagimeli Please see my changes and let me know if you have any questions. Thanks!

_ingest-pipelines/processors/rename.md Outdated Show resolved Hide resolved
_ingest-pipelines/processors/rename.md Outdated Show resolved Hide resolved
_ingest-pipelines/processors/rename.md Outdated Show resolved Hide resolved
vagimeli and others added 3 commits April 17, 2024 09:27
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
@vagimeli vagimeli added 6 - Done but waiting to merge PR: The work is done and ready to merge v2.14.0 and removed 5 - Editorial review PR: Editorial review in progress labels Apr 17, 2024
@hdhalter hdhalter added backport 2.13 PR: Backport label for 2.13 and removed 6 - Done but waiting to merge PR: The work is done and ready to merge v2.14.0 labels Apr 22, 2024
@hdhalter hdhalter merged commit 3ccdf47 into opensearch-project:main Apr 22, 2024
8 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Apr 22, 2024
* Add rename ingest processor

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

* Update rename.md

Signed-off-by: Melissa Vagi <[email protected]>

* Update index-processors.md

Signed-off-by: Melissa Vagi <[email protected]>

* Update rename.md

Signed-off-by: Melissa Vagi <[email protected]>

* Update _ingest-pipelines/processors/rename.md

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>

* Update _ingest-pipelines/processors/rename.md

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>

* Update _ingest-pipelines/processors/rename.md

Co-authored-by: Nathan Bower <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]>
(cherry picked from commit 3ccdf47)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
github-actions bot pushed a commit that referenced this pull request Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.13 PR: Backport label for 2.13
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOC] Add rename ingest processor
4 participants