Skip to content

[ES-8187] ES|QL elastic/logs scale testing challenge#618

Closed
favilo wants to merge 14 commits intoelastic:masterfrom
favilo:esql-scale-testing
Closed

[ES-8187] ES|QL elastic/logs scale testing challenge#618
favilo wants to merge 14 commits intoelastic:masterfrom
favilo:esql-scale-testing

Conversation

@favilo
Copy link
Contributor

@favilo favilo commented Jun 13, 2024

Adding special queries specifically for scale testing ESQL queries

@favilo favilo requested a review from craigtaverner June 13, 2024 23:34
@bpintea bpintea self-requested a review June 17, 2024 10:28
Copy link

@bpintea bpintea left a comment

Choose a reason for hiding this comment

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

I've left some comments mostly concerning queries that might no longer be needed.
I think it'd be great if we only have those queries present that we want timing details about.

@gbanasiak
Copy link
Contributor

@favilo I've modified queries in 3747188 to reduce response time but hard-coded timestamp is problematic. I don't think this should land in master branch.

Copy link
Contributor

@craigtaverner craigtaverner left a comment

Choose a reason for hiding this comment

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

Approved, but I wonder were all four of these really long running? If not, we should not push the iteration count down for fast queries.

@gbanasiak
Copy link
Contributor

I wonder were all four of these really long running?

Yes.

@gbanasiak
Copy link
Contributor

#705 includes the queries from this PR, so I'll rework it.

@favilo favilo closed this Feb 24, 2025
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.

4 participants

Comments