Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

HyperProcess::new documents a 30s callback timeout but waits 60s

Aperta Adatta ai principianti
  • Qualcosa non va?
#294 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
68/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
rust
Ambito
api

Direzione di ricerca

Leggi hyperdb-api/src/process.rs:226 e wait_for_callback alla riga 709, quindi controlla in CI l’avvio lento di hyperd segnalato. Decidi se il contratto previsto è di 30 o 60 secondi, allinea la documentazione e l’implementazione e valuta un const documentato, in modo che non possano divergere; il lavoro è completato quando il timeout pubblico e l’attesa effettiva coincidono.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Summary

HyperProcess::new's doc comment promises a 30-second timeout for Hyper's connect-back callback, but the implementation waits 60 seconds.

The documented contract:

/// - Hyper doesn't connect back within the timeout (30 seconds)

— hyperdb-api/src/process.rs:226

The actual wait, in wait_for_callback:

let timeout = Duration::from_secs(60);

— hyperdb-api/src/process.rs:709

Why it matters

A caller sizing their own timeout or retry budget against the documented 30 seconds will be surprised by a failure path that takes twice as long. It matters most on a slow or loaded host, which is exactly when the callback is slowest and when a caller is most likely to be relying on the documented bound.

The doc comment is the public contract, so this is a defect in one of the two — either the doc understates the wait, or the implementation waits longer than intended.

Fix direction

Decide which value is correct and make the other match:

  • If 60 s is the intended behaviour, update the doc comment.
  • If 30 s is intended, change the constant — but check CI first. hyperd startup has been observed taking "10+ seconds under load" on CI runners, so halving this bound could introduce flakiness on the slowest legs. That argues for correcting the documentation rather than the constant, but the value should be a deliberate choice rather than an accident.

Either way, consider naming it as a const with a doc comment so the two cannot drift again, and referencing that constant from HyperProcess::new's docs instead of restating the number in prose.

Provenance

Noticed incidentally while auditing hyperdb-api's process lifecycle for the shared-daemon design exploration (#293). Verified against main at 6cfe312.

Lingua principale
Rust
Stelle
3
Fork
2
Merge medio
23h 11m
PR unite (30g)
65

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di tableau/hyper-api-rust

Tutte le issue di tableau/hyper-api-rust

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.