hound-search/hound

Exceed limit results

Aberta

#217 aberto em 20 de jun. de 2016

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)JavaScript (606 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (5.470 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Hi guys, as I can see here https://github.com/etsy/hound/blob/203bd0c7601e28de8c4ae4e922bc0b76938af268/index/index.go#L198 - results are just dropped if they exceed limit (which is constant - 5000).

It is possible to return what was collected so far, instead of just dropping everything. Or, alternatively you can let user set max number of results he wants, of course limited by server's limit too.

For me both are better than what we have now, what do you think?

Guia do colaborador

Direção de pesquisa
Investigue a implementação atual em index/index.go por volta da linha 198, onde os resultados são descartados se excederem o limite constante de 5000. Considere implementar uma das opções: (1) modificar o código para retornar resultados parciais até o limite, ou (2) adicionar um limite configurável via entrada do usuário, com um limite máximo do lado do servidor. Avalie qual abordagem está mais alinhada com os objetivos do projeto e discuta com os mantenedores.
Pilha de tecnologia
go
Domain
backend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Go
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60