elastic/elasticsearch

Should reindex serialize partial index and search requests

Open

#22.864 aperta il 30 gen 2017

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)Java (25.882 fork)batch import
:Distributed/Reindex>bugTeam:Distributedhelp wanted

Metriche repository

Star
 (76.700 star)
Metriche merge PR
 (Merge medio 2g) (1000 PR mergiate in 30 g)

Descrizione

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?

Guida contributor

Direzione di ricerca
Esamina la pull request #22691 e la discussione per comprendere l'assunto di serializzazione, poi esamina il codice sorgente di reindex per vedere come vengono costruite e serializzate le richieste parziali di ricerca e indicizzazione. Identifica dove l'assunto viene violato e proponi una correzione per serializzare solo i dati previsti.
Tech stack
java
Dominio
backendapidatabase
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
JavaElasticsearch
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40