Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

On slow networks, deduplication drops requests #1274

Open
TCA-Bot opened this issue Jul 2, 2024 · 6 comments
Open

On slow networks, deduplication drops requests #1274

TCA-Bot opened this issue Jul 2, 2024 · 6 comments
Labels
entry Related to an entry

Comments

@TCA-Bot
Copy link
Collaborator

TCA-Bot commented Jul 2, 2024

"Institute for Advanced Study (IAS)" wird bei der Suche nach "IAS" nicht in den ersten Suchergebnissen angezeigt, obwohl die Abkürzung Teil des Gebäudenamens auf nav.tum.de ist.

@TCA-Bot TCA-Bot added entry Related to an entry webform labels Jul 2, 2024
@CommanderStorm

This comment was marked as outdated.

@AntonMC-Github
Copy link

Wird bei mir jetzt auch angezeigt, keine Ahnung, was da eben nicht ging - vielleicht hat die Suche sich bei meinem langsamen Internet nur nicht aktualisiert, issue kann also gerne weg :)

@CommanderStorm
Copy link
Member

Das issue ist schon valide.

  • Was genau wurde dir denn angezeigt?
  • Wie warst du connected? (ubahn+handy, handy, wifi, ..)

Eventuell muss ich mir mal diese settings genauer anschauen wie sie sich auf connected mobility auswirken.

(momentan: dedupe=defer)

dedupe: avoid fetching same key more than once at a time. Possible options:

  • cancel - cancels existing requests when a new one is made
  • defer - does not make new requests at all if there is a pending request

Eventuell sollte man nach einer gewissen zeit (200-300ms, sonnst zu disruptiv) auch statt alten suchergebnissen einen spinner anzeigen, falls wir den request neu versuchen müssen.
Warscheinlich kann ich den Wert hier auch etwas smarter setzen (das ist momentan bei 5s um daten beim nutzer zu sparen => falls nutzer gutes internet hat sollten wir alle 500ms retryen)

@AntonMC-Github
Copy link

AntonMC-Github commented Jul 2, 2024

Hab's nochmal repliziert bekommen - diesmal sogar mit Screenshot :)

Umgebung ist folgende:
WLAN aus Italien (Ping nach Nürnberg: 29ms) mit 700KBit Down / 30 MBit Up

Ich hab' direkt nach Laden der Website die Suche verwendet.

Wenn ich dann nochmal 10-20 Sekunden gewartet habe und dann nochmal das ganze eingegeben hab' ging's dann

Die Website wurde auch bei der ersten Eingabe schon als fertig geladen angezeigt

@CommanderStorm CommanderStorm changed the title [Entry] [5301]: "Institute for Advanced Study (IAS)" wird bei der Suche nach "IAS" nicht in den ersten Suchergebnissen angezeigt. On slow networks, deduplication drops requests Jul 2, 2024
@CommanderStorm
Copy link
Member

CommanderStorm commented Jul 2, 2024

Super weird.
Can reproduce this by doing (1-3)

image

@CommanderStorm
Copy link
Member

FYI:
als hotfix habe ich dedupe=cancel gesetzt.
Das ist aber nicht optimal, da damit erst alle inflight requests sofort gecancelt werden (anstatt schön zu pipelinen)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
entry Related to an entry
Projects
None yet
Development

No branches or pull requests

3 participants