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.6] Add truncate string processor #3950

Closed
wants to merge 1 commit into from

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport c3fe808 from #3924

* Add truncate string processor

Signed-off-by: Krishna Kondaka <[email protected]>

* Addressed review comments

Signed-off-by: Krishna Kondaka <[email protected]>

* Added check for negative numbers in the config input

Signed-off-by: Krishna Kondaka <[email protected]>

* Fixed checkstyle error

Signed-off-by: Krishna Kondaka <[email protected]>

* Modified to make truncate processor a top level processor, not specific to strings

Signed-off-by: Krishna Kondaka <[email protected]>

* Addressed review comments

Signed-off-by: Krishna Kondaka <[email protected]>

* Updated documentation with correct configuration

Signed-off-by: Krishna Kondaka <[email protected]>

* Fixed typos in the documentation

Signed-off-by: Krishna Kondaka <[email protected]>

* Modified to allow more than one source keys in the config

Signed-off-by: Krishna Kondaka <[email protected]>

* Modified to allow multiple entries under configuration

Signed-off-by: Krishna Kondaka <[email protected]>

---------

Signed-off-by: Krishna Kondaka <[email protected]>
Co-authored-by: Krishna Kondaka <[email protected]>
(cherry picked from commit c3fe808)
Copy link

github-actions bot commented Jan 11, 2024

Unit Test Results

  1 642 files   -    602    1 642 suites   - 602   38m 15s ⏱️ - 15m 16s
  6 484 tests  -    324    6 482 ✔️  -    325  1 💤 ±0  1 +1 
14 048 runs   - 4 363  14 045 ✔️  - 4 363  2 💤  - 1  1 +1 

For more details on these failures, see this check.

Results for commit de7369f. ± Comparison against base commit 210e9ce.

♻️ This comment has been updated with latest results.

@dlvenable
Copy link
Member

@kkondaka , This is a new feature and should not be backported. We will release it in 2.7.

@dlvenable dlvenable closed this Jan 11, 2024
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.

2 participants