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

ledger: CurrenciesIn's overflow-bound comment names takeLocked; foldLocked holds the reservation

Chiusa Adatta ai principianti
  • Qualcosa non va?
#1,178 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à
1/5
Tempo stimato
Meno di un'ora
Idoneità per principianti
92/100
Tipo di issue
Documentazione
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
go
Ambito
backend

Direzione di ricerca

Leggi il commento sui minuti limitati in core/cost/ledger/query.go, quindi verifica la condizione di prenotazione in foldLocked in core/cost/ledger/writer.go. Aggiorna il commento in modo che menzioni foldLocked invece di takeLocked, mantenendo la spiegazione esistente del limite. Il lavoro è completato quando il commento corrisponde all’implementazione e il controllo di revisione C14 va a buon fine.

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

Descrizione

Found by the round-3 strict review of #1153, in a comment that PR's round 2 added.

core/cost/ledger/query.go says the capped minute keeps maxLabelsPerMinute-1 real rows, "Minus one because takeLocked reserves the last slot for overflowKey itself". The reservation is in foldLocked, not takeLocked — core/cost/ledger/writer.go carries if len(w.rows) >= maxLabelsPerMinute-1 in that function.

The number is right and the conclusion it supports (the overflow row is never the only row) holds. Only the function name is wrong — a mechanism stated without checking the symbol resolves, which is the specific mistake grep -rnE '^func +(\([^)]*\) *)?<Name>\b' catches.

Tracked red by the review harness as check C14, which re-derives the bound from writer.go rather than from the comment.

Assisted-By: Claude (Anthropic AI) [email protected]

Lingua principale
Go
Stelle
13
Fork
40
Merge medio
12h 3m
PR unite (30g)
199

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 rossoctl/cortex

Tutte le issue di rossoctl/cortex

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.