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

ESQL: Reeanble another test #100342

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

nik9000
Copy link
Member

@nik9000 nik9000 commented Oct 5, 2023

We'd disabled this one a week ago because EVAL | STATS wasn't properly releasing blocks. It does now so we can have this test back.

We'd disabled this one a week ago because `EVAL | STATS` wasn't properly
releasing blocks. It does now so we can have this test back.
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-ql (Team:QL)

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/elasticsearch-esql (:Query Languages/ES|QL)

Copy link
Member

@dnhatn dnhatn left a comment

Choose a reason for hiding this comment

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

LGTM

@dnhatn dnhatn added the auto-backport-and-merge Automatically create backport pull requests and merge when ready label Oct 5, 2023
@nik9000 nik9000 added the auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Oct 5, 2023
@elasticsearchmachine elasticsearchmachine merged commit e1fa3ef into elastic:main Oct 5, 2023
11 checks passed
@nik9000 nik9000 deleted the heap_attack_reenable2 branch October 5, 2023 14:54
nik9000 added a commit to nik9000/elasticsearch that referenced this pull request Oct 5, 2023
We'd disabled this one a week ago because `EVAL | STATS` wasn't properly
releasing blocks. It does now so we can have this test back.
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.11

elasticsearchmachine pushed a commit that referenced this pull request Oct 5, 2023
We'd disabled this one a week ago because `EVAL | STATS` wasn't properly
releasing blocks. It does now so we can have this test back.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/ES|QL AKA ESQL auto-backport-and-merge Automatically create backport pull requests and merge when ready auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) >non-issue Team:QL (Deprecated) Meta label for query languages team v8.11.0 v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants