Skip to content

Conversation

@idegtiarenko
Copy link
Contributor

@dnhatn and I discussed that this value is a bit too high for small results sets leading to higher than ideal transport request count.
2 should still be sufficient to transmit a bigger results, especially since some Block sizes were optimized

@idegtiarenko idegtiarenko requested a review from dnhatn June 3, 2025 15:42
@idegtiarenko idegtiarenko added >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL v9.1.0 labels Jun 3, 2025
@elasticsearchmachine
Copy link
Collaborator

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

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. Thank you!

@idegtiarenko idegtiarenko added auto-backport Automatically create backport pull requests when merged v8.19.0 labels Jun 4, 2025
@idegtiarenko idegtiarenko merged commit 28f1e59 into elastic:main Jun 4, 2025
18 checks passed
@idegtiarenko idegtiarenko deleted the reduce_concurrent_clients branch June 4, 2025 07:21
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.19

idegtiarenko added a commit to idegtiarenko/elasticsearch that referenced this pull request Jun 4, 2025
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 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