elastic/elasticsearch

SecurityMigrationExecutor and SystemIndexMigrationExecutor should not be persistent tasks

Ouverte

#146 662 ouverte le 17 avr. 2026

Voir sur GitHubGuide contributeur
 (7 commentaires) (0 réaction) (0 personne assignée)Java (25 882 forks)batch import
:Distributed/DistributedTeam:Distributedgood first issue

Métriques du dépôt

Stars
 (76 700 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Both SecurityMigrationExecutor and SystemIndexMigrationExecutor are run-to-completion one-shot migration tasks rather than continuously-running services. They do not "persist" in any meaningful sense, instead they have some external retry mechanism, so it seems that they would better fit as just regular transport actions instead.

Refactoring those task to not be persistent tasks would also get us one step closer to removing the automaticReassignmentOnShutdown() persistent task executor flag (see https://github.com/elastic/elasticsearch/issues/138565) and making this feature default everywhere.

See: https://github.com/elastic/elasticsearch/pull/145753 for more details

Guide contributeur

Direction de recherche
Examinez la PR #145753 référencée pour comprendre le modèle de refactoring attendu, puis examinez l'implémentation de SecurityMigrationExecutor et SystemIndexMigrationExecutor pour identifier comment les convertir de tâches persistantes en actions de transport régulières.
Stack technique
java
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
JavaElasticsearch internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30