hound-search/hound

Exceed limit results

Offen

#217 geöffnet am 20.06.2016

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (606 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (5.470 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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?

Contributor Guide

Research-Richtung
Untersuchen Sie die aktuelle Implementierung in index/index.go um Zeile 198, wo Ergebnisse verworfen werden, wenn sie die konstante Grenze von 5000 überschreiten. Erwägen Sie die Implementierung einer der folgenden Optionen: (1) Ändern Sie den Code, um Teilergebnisse bis zur Grenze zurückzugeben, oder (2) fügen Sie ein konfigurierbares Limit über Benutzereingaben hinzu, mit einer serverseitigen Obergrenze. Bewerten Sie, welcher Ansatz besser mit den Zielen des Projekts übereinstimmt, und besprechen Sie dies mit den Maintainern.
Tech Stack
go
Domain
backend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Go
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60