Skip to content

Conversation

@mashhurs
Copy link
Contributor

@mashhurs mashhurs commented May 6, 2025

  • Introduces a target field as a field reference (mixin validated) where if set result is placed into the target.

  • Simplifies the set extracted values to the event with target logic. Applies setting to target with aggregations similarly with es-input.

  • Mention to target in each fields which can be placed in the target. Docs info fields are placed in target field.


(cherry picked from commit 5abbe49)

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/


Cherry-pick merge conflict files:

  • CHANGELOG.MD: main vs 3.x logs are always different
  • logstash-filter-elasticsearch.gemspec: version conflict
  • filters/elasticsearch/elasticsearch.rb: due to SSL setting obsoletion which also removed the normalize_config_support

* Introduces a target field as a field reference (mixin validated) where if set result is placed into the target.

* Simplifies the set extracted values to the event with target logic. Applies setting to target with aggregations similarly with es-input.

* Mention to target in each fields which can be placed in the target. Docs info fields are placed in target field.

---------

Co-authored-by: Rye Biesemeyer <yaauie@users.noreply.github.com>
(cherry picked from commit 5abbe49)
@mashhurs mashhurs requested a review from yaauie May 6, 2025 20:25
@mashhurs mashhurs changed the title Add a target field support. (#196) [3.x] Add a target field support. (#196) May 6, 2025
@donoghuc donoghuc self-requested a review May 7, 2025 16:53
Copy link
Contributor

@donoghuc donoghuc left a comment

Choose a reason for hiding this comment

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

Looks clean!

@mashhurs mashhurs merged commit d3a1701 into logstash-plugins:3.x May 7, 2025
3 checks passed
@mashhurs mashhurs deleted the target-support-3.x branch May 7, 2025 17:17
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