Skip to content

Prometheus: Build plumbing — widen PromqlCommand.STEP_COLUMN_NAME and add ES|QL deps to prometheus plugin#144301

Merged
felixbarny merged 4 commits intoelastic:mainfrom
felixbarny:promql-query-range-pr1
Mar 19, 2026
Merged

Prometheus: Build plumbing — widen PromqlCommand.STEP_COLUMN_NAME and add ES|QL deps to prometheus plugin#144301
felixbarny merged 4 commits intoelastic:mainfrom
felixbarny:promql-query-range-pr1

Conversation

@felixbarny
Copy link
Copy Markdown
Member

@felixbarny felixbarny commented Mar 16, 2026

Summary

This is PR 1 of 4 in a stacked series that adds the Prometheus query_range endpoint.
See the overview PR #144294 for context.

Two pure infrastructure changes with no behaviour change:

  • PromqlCommand.java: widens STEP_COLUMN_NAME from private to public so the Prometheus plugin can reference it without reflection.
  • prometheus/build.gradle: adds x-pack-esql to extendedPlugins and adds compileOnly / testImplementation deps on x-pack-esql and x-pack-esql-core so the Prometheus plugin can compile against ES|QL types.

Stacking order

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

…s to prometheus plugin

- Widen `PromqlCommand.STEP_COLUMN_NAME` from `private` to `public` so the
  Prometheus plugin can reference it without reflection.
- Add `x-pack-esql` to `extendedPlugins` in prometheus/build.gradle so the
  plugin can depend on ES|QL classes at runtime.
- Add `compileOnly` and `testImplementation` deps on `x-pack-esql` and
  `x-pack-esql-core` so the prometheus plugin can compile against ES|QL types.
@felixbarny felixbarny merged commit 7895fc4 into elastic:main Mar 19, 2026
40 checks passed
@felixbarny felixbarny deleted the promql-query-range-pr1 branch March 19, 2026 06:42
michalborek pushed a commit to michalborek/elasticsearch that referenced this pull request Mar 23, 2026
…s to prometheus plugin (elastic#144301)

- Widen `PromqlCommand.STEP_COLUMN_NAME` from `private` to `public` so the
  Prometheus plugin can reference it without reflection.
- Add `x-pack-esql` to `extendedPlugins` in prometheus/build.gradle so the
  plugin can depend on ES|QL classes at runtime.
- Add `compileOnly` and `testImplementation` deps on `x-pack-esql` and
  `x-pack-esql-core` so the prometheus plugin can compile against ES|QL types.
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