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

[Backport 2.x] Add fingerprint ingest processor #14366

Merged
merged 1 commit into from
Jun 15, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 16c8806 from #13724.

* Add fingerprint ingest processor

Signed-off-by: Gao Binlong <[email protected]>

* Ignore metadata fields

Signed-off-by: Gao Binlong <[email protected]>

* Add sha3-256 hash method

Signed-off-by: Gao Binlong <[email protected]>

* Remove unused code

Signed-off-by: Gao Binlong <[email protected]>

* Add exclude_fields and remove include_all_fields

Signed-off-by: Gao Binlong <[email protected]>

* Modify processor description

Signed-off-by: Gao Binlong <[email protected]>

* Make FingerprintProcessor being final

Signed-off-by: Gao Binlong <[email protected]>

* Optimize error message and check if field name is empty string

Signed-off-by: Gao Binlong <[email protected]>

* Fix yaml test failure

Signed-off-by: Gao Binlong <[email protected]>

* Prepend string length to the field value

Signed-off-by: Gao Binlong <[email protected]>

* Append hash method with version number

Signed-off-by: Gao Binlong <[email protected]>

* Update supported version in yml test file

Signed-off-by: Gao Binlong <[email protected]>

* Add more comment

Signed-off-by: Gao Binlong <[email protected]>

* Prepend hash method to the hash value and add more test cases

Signed-off-by: Gao Binlong <[email protected]>

---------

Signed-off-by: Gao Binlong <[email protected]>
(cherry picked from commit 16c8806)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

❕ Gradle check result for 2dca1b1: UNSTABLE

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

@reta reta merged commit 5466e90 into 2.x Jun 15, 2024
51 checks passed
kkewwei pushed a commit to kkewwei/OpenSearch that referenced this pull request Jul 24, 2024
…ch-project#14366)

* Add fingerprint ingest processor

* Ignore metadata fields

* Add sha3-256 hash method

* Remove unused code

* Add exclude_fields and remove include_all_fields

* Modify processor description

* Make FingerprintProcessor being final

* Optimize error message and check if field name is empty string

* Fix yaml test failure

* Prepend string length to the field value

* Append hash method with version number

* Update supported version in yml test file

* Add more comment

* Prepend hash method to the hash value and add more test cases

---------

(cherry picked from commit 16c8806)

Signed-off-by: Gao Binlong <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: kkewwei <[email protected]>
@dbwiddis dbwiddis deleted the backport/backport-13724-to-2.x branch July 24, 2024 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant