Skip to content

Commit efd41c5

Browse files
Remove FailedToCommitClusterStateException Check (#135846)
Removes a check on `FailedToCommitClusterStateException` with a message "node closed" inside 1QueryableBuiltInRolesSynchronizer .isExpectedFailure1 since this exception was changed to a 1NotMasterException1 inside #135008 Relates to: ES-13061
1 parent c3ae278 commit efd41c5

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/support/QueryableBuiltInRolesSynchronizer.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
import org.elasticsearch.cluster.ClusterStateTaskListener;
2121
import org.elasticsearch.cluster.NotMasterException;
2222
import org.elasticsearch.cluster.SimpleBatchedExecutor;
23-
import org.elasticsearch.cluster.coordination.FailedToCommitClusterStateException;
2423
import org.elasticsearch.cluster.metadata.IndexMetadata;
2524
import org.elasticsearch.cluster.metadata.Metadata;
2625
import org.elasticsearch.cluster.metadata.ProjectMetadata;
@@ -309,8 +308,7 @@ private static boolean isExpectedFailure(final Exception e) {
309308
|| cause instanceof ResourceAlreadyExistsException
310309
|| cause instanceof VersionConflictEngineException
311310
|| cause instanceof DocumentMissingException
312-
|| cause instanceof FailedToMarkBuiltInRolesAsSyncedException
313-
|| (e instanceof FailedToCommitClusterStateException && "node closed".equals(cause.getMessage()));
311+
|| cause instanceof FailedToMarkBuiltInRolesAsSyncedException;
314312
}
315313

316314
private boolean shouldSyncBuiltInRoles(final ClusterState state) {

0 commit comments

Comments
 (0)