Skip to content

Measure limit 10 performance#760

Merged
idegtiarenko merged 2 commits intoelastic:masterfrom
idegtiarenko:limit_10
Mar 20, 2025
Merged

Measure limit 10 performance#760
idegtiarenko merged 2 commits intoelastic:masterfrom
idegtiarenko:limit_10

Conversation

@idegtiarenko
Copy link
Contributor

This adds a track with limit 10 for a sample like queries.

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 @idegtiarenko, this looks good, but to make these queries actually run you'll also have to add the operations to the default challenge, see https://github.com/elastic/rally-tracks/blob/master/esql/challenges/default.json#L59-L65

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.

LGTM, thanks!

@idegtiarenko idegtiarenko merged commit f6a6487 into elastic:master Mar 20, 2025
13 checks passed
@idegtiarenko idegtiarenko deleted the limit_10 branch March 20, 2025 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments