Skip to content

EQL: Fix sequences with conditions involving keys and non-keys #133134

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 9 commits into from
Aug 22, 2025
Merged

EQL: Fix sequences with conditions involving keys and non-keys #133134

merged 9 commits into from
Aug 22, 2025

Conversation

luigidellaquila
Copy link
Contributor

@luigidellaquila luigidellaquila commented Aug 19, 2025 •
edited
Loading

Fixes: #133065

Fixing the case where a filter on a join key cannot be propagated because it also involves other fields, eg.

sequence by foo with maxspan=5s 
[any  where true ] 
[any startswith(foo, bar) ]

@elasticsearchmachine elasticsearchmachine added v9.2.0 Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Aug 19, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@elasticsearchmachine
Copy link
Collaborator

Hi @luigidellaquila, I've created a changelog YAML for you.

astefan
astefan approved these changes Aug 22, 2025
View reviewed changes
Copy link
Contributor

@astefan astefan left a comment

Choose a reason for hiding this comment

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

LGTM

@astefan astefan added v9.1.4 v8.18.6 v8.19.3 v9.0.7 auto-backport Automatically create backport pull requests when merged labels Aug 22, 2025
@luigidellaquila luigidellaquila merged commit 6b1560d into elastic:main Aug 22, 2025
34 checks passed
This was referenced Aug 22, 2025
Merged
Merged
luigidellaquila added a commit to luigidellaquila/elasticsearch that referenced this pull request Aug 22, 2025
@luigidellaquila
EQL: Fix sequences with conditions involving keys and non-keys (elast…
18fff16 
…ic#133134)
This was referenced Aug 22, 2025
Merged
Merged
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
✅ 8.18
✅ 8.19
✅ 9.1
✅ 9.0

luigidellaquila added a commit to luigidellaquila/elasticsearch that referenced this pull request Aug 22, 2025
@luigidellaquila
EQL: Fix sequences with conditions involving keys and non-keys (elast…
af7ce8d 
…ic#133134)
luigidellaquila added a commit to luigidellaquila/elasticsearch that referenced this pull request Aug 22, 2025
@luigidellaquila
EQL: Fix sequences with conditions involving keys and non-keys (elast…
8ac3cf8 
…ic#133134)
luigidellaquila added a commit to luigidellaquila/elasticsearch that referenced this pull request Aug 22, 2025
@luigidellaquila
EQL: Fix sequences with conditions involving keys and non-keys (elast…
1f6c505 
…ic#133134)
elasticsearchmachine pushed a commit that referenced this pull request Aug 22, 2025
@luigidellaquila
EQL: Fix sequences with conditions involving keys and non-keys (#133134…
f60af12 
…) (#133383)
elasticsearchmachine pushed a commit that referenced this pull request Aug 22, 2025
@luigidellaquila
EQL: Fix sequences with conditions involving keys and non-keys (#133134…
ec7e5e8 
…) (#133384)
elasticsearchmachine pushed a commit that referenced this pull request Aug 22, 2025
@luigidellaquila
EQL: Fix sequences with conditions involving keys and non-keys (#133134…
ac11f2e 
…) (#133382)
elasticsearchmachine pushed a commit that referenced this pull request Aug 22, 2025
@luigidellaquila
EQL: Fix sequences with conditions involving keys and non-keys (#133134…
b86ea84 
…) (#133385)
pabloem pushed a commit to pabloem/elasticsearch that referenced this pull request Aug 22, 2025
@luigidellaquila @pabloem
EQL: Fix sequences with conditions involving keys and non-keys (elast…
7f51ad4 
…ic#133134)
@w0rk3r w0rk3r mentioned this pull request Sep 1, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@astefan astefan astefan approved these changes

Assignees

No one assigned

Labels

:Analytics/EQL EQL querying auto-backport Automatically create backport pull requests when merged >bug Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.18.6 v8.19.3 v9.0.7 v9.1.4 v9.2.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

EQL: sequence doesn't match join key when using startswith~() in a filter

3 participants

@luigidellaquila @elasticsearchmachine @astefan