Skip to content

Add ESQL telemetry collection#119474

Merged
smalyshev merged 35 commits intoelastic:mainfrom
smalyshev:esql-telemetry
Jan 2, 2025
Merged

Add ESQL telemetry collection#119474
smalyshev merged 35 commits intoelastic:mainfrom
smalyshev:esql-telemetry

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
@github-actions
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.

Co-authored-by: Marci W <333176+marciw@users.noreply.github.com>
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
@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

4 participants