elastic/elasticsearch

Should reindex serialize partial index and search requests

Open

#22.864 aberto em 30 de jan. de 2017

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)Java (25.882 forks)batch import
:Distributed/Reindex>bugTeam:Distributedhelp wanted

Métricas do repositório

Stars
 (76.700 stars)
Métricas de merge de PR
 (Mesclagem média 2d) (1.000 fundiu PRs em 30d)

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?

Guia do colaborador

Direção de pesquisa
Investigue o pull request #22691 e a discussão para entender a suposição de serialização, depois examine o código fonte de reindex para ver como as solicitações parciais de pesquisa e índice são construídas e serializadas. Identifique onde a suposição é violada e proponha uma correção para serializar apenas os dados esperados.
Pilha de tecnologia
java
Domain
backendapidatabase
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
JavaElasticsearch
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40