Skipped-partition stderr wording omits that non-session skipped rows stay sink-exportable

Aperta
  • Qualcosa non va?
#2,048 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
45/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
javascript
Ambito
cli

Direzione di ricerca

Read src/core/commands/purge.js:128-134 and the related session-target handling at lines 95-96; check the existing purge tests or documentation for how stderr wording is verified. Done means the stderr message or purge documentation explains that skipped rows for non-session targets remain exportable until a successful rerun, with a test or doc check pinning that wording.

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

Descrizione

Source: #2043 at 94bd66662c08b4ce2fb352978bc93ac45c412532
Location: src/core/commands/purge.js:128-134
Severity: minor

Evidence

The durable purge fence is written only for session targets (src/core/commands/purge.js:95-96, createSessionPurgeStore), so for subtree/ignored/all targets a skipped partition's surviving rows remain query-visible and exportable to configured sinks until a rerun succeeds. The stderr line says only 'may still hold matching rows', which is true but understates the export consequence for those targets.

Why deferral is safe

The message truthfully states non-deletion and the run exits 1, so a user is told the rows survive and chained flows (the privacy skill invokes hyp purge with && chaining) stop before a sync; the export exposure is a consequence of surviving rows, not a claim the message contradicts. Behavior is strictly better than pre-PR, where the wholesale abort left every partition's rows equally exportable with the same wording absent.

Acceptance condition

The stderr line (or the purge documentation) states that for non-session targets skipped partitions' rows remain exportable until a rerun succeeds, with a test or doc check pinning the wording.

Review preference; backlog only. A human may explicitly delegate it with neutral:fix.

Lingua principale
JavaScript
Stelle
11
Fork
0
Merge medio
2h 49m
PR unite (30g)
302

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

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 hyparam/hypaware

Tutte le issue di hyparam/hypaware

Issue simili

Altre issue su JavaScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.