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 copy ingest processor #6294

Merged
merged 13 commits into from
Feb 5, 2024
Merged

Add copy ingest processor #6294

merged 13 commits into from
Feb 5, 2024

Conversation

gaobinlong
Copy link
Contributor

@gaobinlong gaobinlong commented Jan 30, 2024

Description

This PR documents the usage of the new added copy ingest processor.

Issues Resolved

Fixes #6147

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.

@gaobinlong
Copy link
Contributor Author

Hi @vagimeli, please help to review this PR, thanks!

@hdhalter hdhalter added 4 - Doc review PR: Doc review in progress release-notes PR: Include this PR in the automated release notes v2.12.0 labels Jan 30, 2024
_ingest-pipelines/processors/copy.md Outdated Show resolved Hide resolved
_ingest-pipelines/processors/copy.md Outdated Show resolved Hide resolved
_ingest-pipelines/processors/copy.md Outdated Show resolved Hide resolved
@vagimeli
Copy link
Contributor

Hi @vagimeli, please help to review this PR, thanks!

@gaobinlong Thank you for putting up the PR. Please see my edits for your approval. I'll move this into editorial review.

_ingest-pipelines/processors/copy.md Outdated Show resolved Hide resolved
@vagimeli vagimeli added 5 - Editorial review PR: Editorial review in progress and removed 4 - Doc review PR: Doc review in progress labels Jan 31, 2024
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.

@vagimeli Just a couple of minor changes. Thanks!

_ingest-pipelines/processors/copy.md Outdated Show resolved Hide resolved
_ingest-pipelines/processors/copy.md Outdated Show resolved Hide resolved
@natebower natebower removed the 5 - Editorial review PR: Editorial review in progress label Feb 1, 2024
@natebower
Copy link
Collaborator

@gaobinlong Would you mind making these two commits?

@hdhalter hdhalter changed the title [Doc] Add copy ingest processor Add copy ingest processor Feb 1, 2024
vagimeli and others added 2 commits February 2, 2024 13:33
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 the 3 - Done Issue is done/complete label Feb 2, 2024
@hdhalter hdhalter merged commit 47013f9 into opensearch-project:main Feb 5, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Done Issue is done/complete release-notes PR: Include this PR in the automated release notes v2.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOC] Add copy ingest processor
4 participants