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 fingerprint processor #7631

Merged
merged 13 commits into from
Jul 9, 2024
Merged

Conversation

gaobinlong
Copy link
Contributor

@gaobinlong gaobinlong commented Jul 4, 2024

Description

Add the documentation of fingerprint processor which will be released in 2.16.0.

Issues Resolved

Resolves #7191

Version

2.16.0

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: gaobinlong <[email protected]>
@gaobinlong
Copy link
Contributor Author

@vagimeli , please help to review this PR, thank you!

@hdhalter hdhalter added 4 - Doc review PR: Doc review in progress release-notes PR: Include this PR in the automated release notes v2.16.0 labels Jul 5, 2024
Signed-off-by: Melissa Vagi <[email protected]>
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 @gaobinlong LGTM with noted changes. Thanks!

_ingest-pipelines/processors/fingerprint.md Outdated Show resolved Hide resolved
_ingest-pipelines/processors/fingerprint.md Outdated Show resolved Hide resolved
_ingest-pipelines/processors/fingerprint.md Outdated Show resolved Hide resolved
_ingest-pipelines/processors/fingerprint.md Outdated Show resolved Hide resolved
_ingest-pipelines/processors/fingerprint.md Outdated Show resolved Hide resolved
_ingest-pipelines/processors/index-processors.md Outdated Show resolved Hide resolved
vagimeli and others added 6 commits July 9, 2024 09:14
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]>
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]>
Copy link
Collaborator

@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 completed.

@vagimeli
Copy link
Collaborator

vagimeli commented Jul 9, 2024

@vagimeli @gaobinlong LGTM with noted changes. Thanks!

Changes accepted. Thank you for prompt review!

@vagimeli vagimeli merged commit 1502ce1 into opensearch-project:main Jul 9, 2024
5 checks passed
@hdhalter hdhalter added 3 - Done Issue is done/complete and removed 4 - Doc review PR: Doc review in progress labels Jul 15, 2024
leanneeliatra pushed a commit to leanneeliatra/opensearch-documentation-website-forl that referenced this pull request Jul 24, 2024
* Add fingerprint processor

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

* Completed doc review

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

* Update _ingest-pipelines/processors/fingerprint.md

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

* Update _ingest-pipelines/processors/fingerprint.md

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

* Update nav order

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

* Update _ingest-pipelines/processors/fingerprint.md

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

* Update _ingest-pipelines/processors/fingerprint.md

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

* Update _ingest-pipelines/processors/fingerprint.md

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

* Update _ingest-pipelines/processors/fingerprint.md

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

* Update _ingest-pipelines/processors/fingerprint.md

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

* Update _ingest-pipelines/processors/index-processors.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]>
Signed-off-by: [email protected] <[email protected]>
sandervandegeijn pushed a commit to sandervandegeijn/documentation-website that referenced this pull request Jul 30, 2024
* Add fingerprint processor

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

* Completed doc review

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

* Update _ingest-pipelines/processors/fingerprint.md

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

* Update _ingest-pipelines/processors/fingerprint.md

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

* Update nav order

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

* Update _ingest-pipelines/processors/fingerprint.md

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

* Update _ingest-pipelines/processors/fingerprint.md

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

* Update _ingest-pipelines/processors/fingerprint.md

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

* Update _ingest-pipelines/processors/fingerprint.md

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

* Update _ingest-pipelines/processors/fingerprint.md

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

* Update _ingest-pipelines/processors/index-processors.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]>
Signed-off-by: Sander van de Geijn <[email protected]>
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.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOC] Add fingerprint ingest processor
4 participants