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

[Profiling] Don't use synthetic source for symbolization queues #126278

Merged

Conversation

rockdaboot
Copy link
Contributor

Using synthetic source for the ephemeral symbolization queue indices profiling-sq-leafframes and profiling-sq-executables isn't needed, and even consumes CPU cycles unnecessarily when reading data.

A secondary issue is a side-effect of using synthetic source: in responses, the field names inside _source are "nested". If synthetic source is disabled, these field names are "dotted".
Since ES 8.17, synthetic source is automatically disabled outside enterprise subscriptions.
This forces us to maintain two code pathes when parsing responses.
Switching away from using synthetic source allows having only one code path for parsing.

@rockdaboot rockdaboot added >bug :UniversalProfiling/Application Elastic Universal Profiling REST APIs and infrastructure v9.1.0 labels Apr 4, 2025
@rockdaboot rockdaboot requested a review from christos68k April 4, 2025 10:36
@rockdaboot rockdaboot self-assigned this Apr 4, 2025
@elasticsearchmachine elasticsearchmachine added the Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team label Apr 4, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@elasticsearchmachine elasticsearchmachine added the external-contributor Pull request authored by a developer outside the Elasticsearch team label Apr 4, 2025
@rockdaboot rockdaboot merged commit e85a38b into elastic:main Apr 4, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug external-contributor Pull request authored by a developer outside the Elasticsearch team Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team :UniversalProfiling/Application Elastic Universal Profiling REST APIs and infrastructure v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants