Skip to content

Fork to WRITE thread before failing shard in updateCheckPoints #87458

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

Conversation

DaveCTurner
Copy link
Contributor

Failing a shard may block on IO so must not happen on a transport worker
thread. With this commit we use a WRITE thread to handle shard
failures caused by exceptions thrown within updateCheckPoints.

Closes #87094

Failing a shard may block on IO so must not happen on a transport worker
thread. With this commit we use a `WRITE` thread to handle shard
failures caused by exceptions thrown within `updateCheckPoints`.

Closes elastic#87094
@DaveCTurner DaveCTurner added >bug :Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. v7.17.5 v8.4.0 v8.3.1 labels Jun 7, 2022
@elasticmachine elasticmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label Jun 7, 2022
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

@elasticsearchmachine
Copy link
Collaborator

Hi @DaveCTurner, I've created a changelog YAML for you.

@DaveCTurner DaveCTurner merged commit 2f097ba into elastic:master Jun 8, 2022
@DaveCTurner DaveCTurner deleted the 2022-06-07-updateCheckPoints-fail-shard-async branch June 8, 2022 07:44
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Jun 8, 2022
…ic#87458)

Failing a shard may block on IO so must not happen on a transport worker
thread. With this commit we use a `WRITE` thread to handle shard
failures caused by exceptions thrown within `updateCheckPoints`.

Closes elastic#87094
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Jun 8, 2022
…ic#87458)

Failing a shard may block on IO so must not happen on a transport worker
thread. With this commit we use a `WRITE` thread to handle shard
failures caused by exceptions thrown within `updateCheckPoints`.

Closes elastic#87094
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
7.17
8.3

elasticsearchmachine pushed a commit that referenced this pull request Jun 8, 2022
… (#87495)

Failing a shard may block on IO so must not happen on a transport worker
thread. With this commit we use a `WRITE` thread to handle shard
failures caused by exceptions thrown within `updateCheckPoints`.

Closes #87094
elasticsearchmachine pushed a commit that referenced this pull request Jun 8, 2022
… (#87496)

Failing a shard may block on IO so must not happen on a transport worker
thread. With this commit we use a `WRITE` thread to handle shard
failures caused by exceptions thrown within `updateCheckPoints`.

Closes #87094
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. v7.17.5 v8.3.1 v8.4.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] CreateIndexIT testCreateAndDeleteIndexConcurrently failing
4 participants