Skip to content

Conversation

smalyshev
Copy link
Contributor

@smalyshev smalyshev commented Jan 2, 2025

Adds _esql section to ccs metrics at _cluster/stats. Only available when any ES|QL queries have been performed.

Taking over from #117282, see more comments there.

@smalyshev smalyshev requested a review from quux00 January 2, 2025 18:22
Copy link
Contributor

github-actions bot commented Jan 2, 2025

Documentation preview:

@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Jan 2, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine elasticsearchmachine removed the needs:triage Requires assignment of a team area label label Jan 2, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@smalyshev smalyshev added needs:triage Requires assignment of a team area label v8.18.0 auto-backport Automatically create backport pull requests when merged Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) and removed Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) needs:triage Requires assignment of a team area label labels Jan 2, 2025
Copy link
Contributor

@marciw marciw left a comment

Choose a reason for hiding this comment

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

Small drive-by style/clarity comments for stats.asciidoc only

Copy link
Contributor

@quux00 quux00 left a comment

Choose a reason for hiding this comment

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

Re-approval after Close/Reopen of PR.

Copy link
Contributor

@marciw marciw left a comment

Choose a reason for hiding this comment

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

docs LGTM

@smalyshev smalyshev merged commit 0292905 into elastic:main Jan 2, 2025
16 checks passed
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 119474

smalyshev added a commit to smalyshev/elasticsearch that referenced this pull request Jan 2, 2025
* Add ESQL telemetry collection

(cherry picked from commit 0292905)

# Conflicts:
#	server/src/main/java/org/elasticsearch/TransportVersions.java
elasticsearchmachine pushed a commit that referenced this pull request Jan 2, 2025
* Add ESQL telemetry collection

(cherry picked from commit 0292905)

# Conflicts:
#	server/src/main/java/org/elasticsearch/TransportVersions.java
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 >enhancement Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.18.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants