Skip to content

Prometheus label values API: add response listener#144957

Merged
felixbarny merged 5 commits intoelastic:mainfrom
felixbarny:prometheus-label-values-api-response-listener
Mar 27, 2026
Merged

Prometheus label values API: add response listener#144957
felixbarny merged 5 commits intoelastic:mainfrom
felixbarny:prometheus-label-values-api-response-listener

Conversation

@felixbarny
Copy link
Copy Markdown
Member

@felixbarny felixbarny commented Mar 25, 2026

Adds PrometheusLabelValuesResponseListener, which converts an EsqlQueryResponse into the Prometheus label values JSON format, handling prefix stripping, truncation detection, and unknown-column errors. It is covered by unit tests in PrometheusLabelValuesResponseListenerTests.

This is part of a stack implementing the Prometheus label values API endpoint:

Stacking order

Converts an EsqlQueryResponse into the Prometheus label values JSON format,
handling prefix stripping, truncation detection, and unknown-column errors.
@felixbarny felixbarny self-assigned this Mar 25, 2026
@elasticsearchmachine elasticsearchmachine added Team:StorageEngine external-contributor Pull request authored by a developer outside the Elasticsearch team labels Mar 25, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

Copy link
Copy Markdown
Member

@kkrik-es kkrik-es left a comment

Choose a reason for hiding this comment

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

Just a comment about testing coverage, please take care of it before merging.

@felixbarny felixbarny requested a review from kkrik-es March 26, 2026 18:12
@felixbarny felixbarny merged commit 9097fe8 into elastic:main Mar 27, 2026
40 checks passed
@felixbarny felixbarny deleted the prometheus-label-values-api-response-listener branch March 27, 2026 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external-contributor Pull request authored by a developer outside the Elasticsearch team >non-issue :StorageEngine/PromQL PromQL support for Elastic Team:StorageEngine v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants