elastic/elasticsearch

Should reindex serialize partial index and search requests

Open

#22 864 ouverte le 30 janv. 2017

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)Java (25 882 forks)batch import
:Distributed/Reindex>bugTeam:Distributedhelp wanted

Métriques du dépôt

Stars
 (76 700 stars)
Métriques de merge PR
 (Merge moyen 2j) (1 000 PRs mergées en 30 j)

Description

As part of https://github.com/elastic/elasticsearch/pull/22691#discussion_r98434085, we noticed some behavior that deviates from the normal assumptions of serializing requests when using reindex. @javanna said:

we always make the assumption that serialization relies on the request being valid according to validate I think.

Based on that statement (valid means validate passes), the way reindex serializes a search request and index request violates the assumption. Should we consider removing the use of these partial requests and just serialize the data we expect for reindex?

Guide contributeur

Direction de recherche
Examinez la pull request #22691 et la discussion pour comprendre l'hypothèse de sérialisation, puis examinez le code source de reindex pour voir comment les requêtes partielles de recherche et d'index sont construites et sérialisées. Identifiez où l'hypothèse est violée et proposez une correction pour sérialiser uniquement les données attendues.
Stack technique
java
Domaine
backendapidatabase
Type d'issue
Bug
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.
Une demi journée
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
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40