Skip to content

ESQL: Revert parallel loading in MultiClusterSpecIT#134738

Merged
elasticsearchmachine merged 1 commit intoelastic:mainfrom
astefan:134086_revert
Sep 15, 2025
Merged

ESQL: Revert parallel loading in MultiClusterSpecIT#134738
elasticsearchmachine merged 1 commit intoelastic:mainfrom
astefan:134086_revert

Conversation

@astefan
Copy link
Copy Markdown
Contributor

@astefan astefan commented Sep 15, 2025

Workarounds #134736 by reverting PR #134086
This is not a fix, but an attempt to quiet down CI while we search for the reason of the failure.

@astefan astefan requested review from dnhatn and nik9000 September 15, 2025 16:57
@astefan astefan added >test Issues or PRs that are addressing/adding tests :Analytics/ES|QL AKA ESQL v9.2.0 labels Sep 15, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Sep 15, 2025
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Member

@dnhatn dnhatn left a comment

Choose a reason for hiding this comment

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

I am trying to understand why this caused CI issues, but reverting the change helps stabilize CI. Thanks, Andrei!

@astefan astefan added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Sep 15, 2025
@elasticsearchmachine elasticsearchmachine merged commit 207787a into elastic:main Sep 15, 2025
35 checks passed
@astefan astefan deleted the 134086_revert branch September 15, 2025 17:54
mridula-s109 pushed a commit to mridula-s109/elasticsearch that referenced this pull request Sep 17, 2025
Workarounds elastic#134736 by
reverting PR elastic#134086 This
is not a fix, but an attempt to quiet down CI while we search for the
reason of the failure.
gmjehovich pushed a commit to gmjehovich/elasticsearch that referenced this pull request Sep 18, 2025
Workarounds elastic#134736 by
reverting PR elastic#134086 This
is not a fix, but an attempt to quiet down CI while we search for the
reason of the failure.
craigtaverner added a commit to craigtaverner/elasticsearch that referenced this pull request Oct 20, 2025
craigtaverner added a commit that referenced this pull request Nov 3, 2025
…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 added a commit to craigtaverner/elasticsearch that referenced this pull request Nov 3, 2025
elasticsearchmachine pushed a commit that referenced this pull request Nov 3, 2025
This reverts commit 4d3e27f. And then
re-applies the revert to
#134738, so we only get an
effective revert of 
#134511.
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-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) 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

Development

Successfully merging this pull request may close these issues.

4 participants