Skip to content

Add query heads priority to SliceQueue#133245

Merged
dnhatn merged 4 commits intoelastic:mainfrom
dnhatn:slice-queue-query-tags
Aug 22, 2025
Merged

Add query heads priority to SliceQueue#133245
dnhatn merged 4 commits intoelastic:mainfrom
dnhatn:slice-queue-query-tags

Conversation

@dnhatn
Copy link
Member

@dnhatn dnhatn commented Aug 20, 2025

With query and tags, SliceQueue will contain more slices (see #132512). This change introduces an additional priority for query heads, allowing Drivers to pull slices from the same query and segment first. This minimizes the overhead of switching between queries and segments.

Relates #132774

@dnhatn dnhatn force-pushed the slice-queue-query-tags branch from 014178a to 3881830 Compare August 20, 2025 17:15
@dnhatn dnhatn changed the title Add queryHeads priority to SliceQueue Aug 20, 2025
@dnhatn dnhatn requested a review from nik9000 August 20, 2025 17:19
@dnhatn dnhatn marked this pull request as ready for review August 20, 2025 17:19
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Aug 20, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

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

@dnhatn
Copy link
Member Author

dnhatn commented Aug 22, 2025

@dnhatn
Copy link
Member Author

dnhatn commented Aug 22, 2025

Thanks Nik!

@dnhatn dnhatn merged commit 7d678a3 into elastic:main Aug 22, 2025
34 checks passed
@dnhatn dnhatn deleted the slice-queue-query-tags branch August 22, 2025 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL >enhancement Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.2.0

3 participants