Skip to content

Conversation

@mashhurs
Copy link
Contributor

@mashhurs mashhurs commented Jun 6, 2025

Cherry picking #233 and updating branch/version specific changelog and version.

ES|QL support:

  • introduces query_type params, accepts dsl or esql option.
  • adds ES|QL executor to execute ESQL query and parse/map response to event validations
  • make sure LS (8.17.4+) supports ES|QL (new elasticsearch-ruby client)
  • make sure connected ES is greater than 8.11+
  • query isn't empty or meaningful that starts with command syntax
  • if query_type is esql, make sure we accept meaningful inputs and do not allow response_type, index, etc.. DSL related params
  • informing if query isn't using METADATA which adds _id, _version to the response entries
  • informing ineffective params such as size, search_api, target if users configure ES|QL results field names in a dotted format. The plugin reproduces nested (example {a.b.c: 'val'} => {'a':{'b':{'c':'val'}}})

- introduces query_type params, accepts dsl or esql option.
- adds ES|QL executor to execute ESQL query and parse/map response to event
validations
- make sure LS (8.17.4+) supports ES|QL (new elasticsearch-ruby client)
- make sure connected ES is greater than 8.11+
- query isn't empty or meaningful that starts with command syntax
- if query_type is esql, make sure we accept meaningful inputs and do not allow response_type, index, etc.. DSL related params
- informing if query isn't using METADATA which adds _id, _version to the response entries
- informing ineffective params such as size, search_api, target if users configure
ES|QL results field names in a dotted format. The plugin reproduces nested (example {a.b.c: 'val'} => {'a':{'b':{'c':'val'}}})
Copy link
Contributor

@robbavey robbavey left a comment

Choose a reason for hiding this comment

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

lgtm

@mashhurs mashhurs merged commit 79f9f43 into logstash-plugins:4.x Jun 6, 2025
2 of 3 checks passed
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