hound-search/hound

Exceed limit results

Aperta

#217 aperta il 20 giu 2016

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (606 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (5470 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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?

Guida contributor

Direzione di ricerca
Indaga l'implementazione corrente in index/index.go intorno alla riga 198, dove i risultati vengono scartati se superano il limite costante di 5000. Considera di implementare una delle seguenti opzioni: (1) modificare il codice per restituire risultati parziali fino al limite, oppure (2) aggiungere un limite configurabile tramite input utente, con un cap lato server. Valuta quale approccio sia più in linea con gli obiettivi del progetto e discuti con i maintainer.
Tech stack
go
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60