Skip to content

Conversation

@idegtiarenko
Copy link
Contributor

Lets set ShardId on NoShardAvailableActionException.
This should help with troubleshooting the issues when this exception is logged/presented outside of the response map or without any other context.

@idegtiarenko idegtiarenko requested review from dnhatn and nik9000 April 17, 2025 08:46
@elasticsearchmachine elasticsearchmachine added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch labels Apr 17, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search-relevance (Team:Search Relevance)

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. Thanks @idegtiarenko.

@idegtiarenko idegtiarenko merged commit 7e5dafa into elastic:main Apr 22, 2025
17 checks passed
@idegtiarenko idegtiarenko deleted the keep_shard_id branch April 22, 2025 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL >non-issue :Search Relevance/Analysis How text is split into tokens Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants