opensearch-project/OpenSearch

[Concurrent Segment Search] Change ConcurrentSearchTasksIT to use custom slice count

Ouverte

#9 317 ouverte le 14 août 2023

Voir sur GitHubGuide contributeur
 (9 commentaires) (1 réaction) (1 personne assignée)Java (1 505 forks)batch import
:testenhancementgood first issue

Métriques du dépôt

Stars
 (8 123 étoiles)
Métriques de merge PR
 (Merge moyen 5j 9h) (266 PRs mergées en 30 j)

Description

Now that #9107 and #8884 have been merged, we should fix ConcurrentSearchTasksIT to not hard code the segment count and instead validate against the customer slice count.

Ref: https://github.com/opensearch-project/OpenSearch/blob/main/server/src/internalClusterTest/java/org/opensearch/action/admin/cluster/node/tasks/ConcurrentSearchTasksIT.java#L111

Guide contributeur

Direction de recherche
Tout d'abord, examinez les PR référencés (#9107 et #8884) pour comprendre la fonctionnalité de nombre de tranches personnalisé. Ensuite, examinez le fichier de test à la ligne spécifiée dans server/src/internalClusterTest/java/org/opensearch/action/admin/cluster/node/tasks/ConcurrentSearchTasksIT.java pour voir le nombre de segments actuellement codé en dur. Le test doit être mis à jour pour utiliser le nombre de tranches personnalisé de la configuration. Assurez vous que le changement s'aligne avec la logique de test existante et ne casse pas d'autres tests.
Stack technique
java
Domaine
backendsearch
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.
3
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
JavaOpenSearchtesting
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70