Skip to content

Conversation

@przemekwitek
Copy link
Contributor

Add assertions about CATEGORIZE ES|QL function usage in ES|QL telemetry for both unit tests and yaml tests.

@przemekwitek przemekwitek added >test Issues or PRs that are addressing/adding tests :ml Machine learning :Analytics/ES|QL AKA ESQL labels Apr 2, 2025
@elasticsearchmachine elasticsearchmachine added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.1.0 Team:ML Meta label for the ML team labels Apr 2, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/ml-core (Team:ML)

@elasticsearchmachine
Copy link
Collaborator

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

@luigidellaquila luigidellaquila added the test-release Trigger CI checks against release build label Apr 2, 2025
@luigidellaquila luigidellaquila self-requested a review April 2, 2025 09:33
Copy link
Contributor

@luigidellaquila luigidellaquila left a comment

Choose a reason for hiding this comment

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

Thanks @przemekwitek, LGTM.
I added test-release label to make sure the PR also passes the release test (we had problems with this test in the past)

@przemekwitek przemekwitek force-pushed the categorize_telemetry branch from 83380bd to 1b8d203 Compare April 2, 2025 12:41
@przemekwitek
Copy link
Contributor Author

run elasticsearch-ci

@przemekwitek przemekwitek merged commit 062d580 into elastic:main Apr 3, 2025
16 of 19 checks passed
@przemekwitek przemekwitek deleted the categorize_telemetry branch April 3, 2025 11:19
andreidan pushed a commit to andreidan/elasticsearch that referenced this pull request Apr 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL :ml Machine learning Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) Team:ML Meta label for the ML team >test Issues or PRs that are addressing/adding tests test-release Trigger CI checks against release build v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants