Skip to content

Revert "ESQL: Speed up MultiClusterSpecIT start up by 40% by parallelizing index creation (#134090) #134511

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
julian-elastic merged 1 commit into from
Sep 10, 2025
Merged

Revert "ESQL: Speed up MultiClusterSpecIT start up by 40% by parallelizing index creation (#134090) #134511

julian-elastic merged 1 commit into from
Sep 10, 2025

Conversation

@julian-elastic
Copy link
Contributor

@julian-elastic julian-elastic commented Sep 10, 2025 •
edited
Loading

This reverts commit 93d0896 merged with #134090
Unfortunately, it caused a flaky test found by #134414

Closes #134414

@elasticsearchmachine elasticsearchmachine added v9.2.0 needs:triage Requires assignment of a team area label labels Sep 10, 2025
@julian-elastic julian-elastic self-assigned this Sep 10, 2025
@julian-elastic julian-elastic added :Analytics/ES|QL AKA ESQL Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests and removed needs:triage Requires assignment of a team area label labels Sep 10, 2025
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Sep 10, 2025

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

@julian-elastic julian-elastic merged commit d915090 into elastic:main Sep 10, 2025
34 checks passed
@astefan astefan mentioned this pull request Sep 17, 2025
Open
@craigtaverner craigtaverner mentioned this pull request Oct 19, 2025
Merged
3 tasks
craigtaverner added a commit to craigtaverner/elasticsearch that referenced this pull request Oct 20, 2025
@craigtaverner
Reapply "ESQL: Speed up MultiClusterSpecIT start up by 40% by paralle…
15c6458 
…lizing index creation (elastic#134090)" (elastic#134511)

This reverts commit d915090.
craigtaverner added a commit that referenced this pull request Nov 3, 2025
@craigtaverner
Re-enable some performance updates to ES|QL spec tests after resilien…
4d3e27f 
…ce improvement to EsqlSpecTestCase (#136781)

* Revert "Revert PR #134086 (#134738)"- This reverts commit 207787a.
* Reapply "ESQL: Speed up MultiClusterSpecIT start up by 40% by parallelizing index creation (#134090)" (#134511) - This reverts commit d915090.
@craigtaverner craigtaverner mentioned this pull request Nov 3, 2025
Merged
elasticsearchmachine pushed a commit that referenced this pull request Nov 3, 2025
@craigtaverner
Revert "Re-enable some performance updates to ES|QL" (#136781) (#137520)
ebca69f 
This reverts commit 4d3e27f. And then
re-applies the revert to
#134738, so we only get an
effective revert of 
#134511.
This was referenced Nov 6, 2025
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dnhatn dnhatn dnhatn approved these changes

@astefan astefan Awaiting requested review from astefan

Assignees

@julian-elastic julian-elastic

Labels

:Analytics/ES|QL AKA ESQL Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.2.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[CI] MultiClusterSpecIT class failing

3 participants

@julian-elastic @elasticsearchmachine @dnhatn