Skip to content

Conversation

@smalyshev
Copy link
Contributor

It is no longer correct that isPartial is the same as query being stopped, so we should separate these two.

@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Feb 14, 2025
@smalyshev smalyshev added the auto-backport Automatically create backport pull requests when merged label Feb 14, 2025
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.

@smalyshev smalyshev force-pushed the partial-stopped-separation branch from e292071 to bd2d7db Compare February 15, 2025 00:20
@smalyshev smalyshev enabled auto-merge (squash) February 15, 2025 00:20
@smalyshev smalyshev merged commit bb9c2c3 into elastic:main Feb 15, 2025
18 checks passed
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.x

elasticsearchmachine pushed a commit that referenced this pull request Feb 15, 2025
* Separate stop flag and is_partial (#122654)

* fix test
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 Automatically create backport pull requests when merged >non-issue >refactoring Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants