Skip to content

Re-enable test and add logging #126193

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.

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

Merged
merged 2 commits into from
Apr 4, 2025
Merged

Re-enable test and add logging #126193

merged 2 commits into from
Apr 4, 2025

Conversation

idegtiarenko
Copy link
Contributor

This enables a test and adds a logging around shard movements so that we could confirm it is causing issues.

Related to: #125947

@idegtiarenko idegtiarenko added >test Issues or PRs that are addressing/adding tests Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL v9.1.0 labels Apr 3, 2025
@idegtiarenko idegtiarenko requested a review from dnhatn April 3, 2025 09:44
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
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.

LGTM.

# Conflicts:
#	muted-tests.yml
@idegtiarenko idegtiarenko merged commit 5a865a4 into elastic:main Apr 4, 2025
17 checks passed
@idegtiarenko idegtiarenko deleted the 125947 branch April 4, 2025 07:17
andreidan pushed a commit to andreidan/elasticsearch that referenced this pull request Apr 9, 2025
arteam added a commit to arteam/elasticsearch that referenced this pull request Apr 14, 2025
The test was fixed and unmuted in elastic#126113, but then muted back by mistake in elastic#126193
arteam added a commit that referenced this pull request Apr 22, 2025
The test was fixed and unmuted in #126113, but then muted back by mistake in #126193
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants