Feature request: lock the data directory so a second PGlite in another process fails fast instead of corrupting it

Aperta
  • Qualcosa non va?
#1,106 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
52/100
Tipo di issue
Funzionalità
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
node.js, postgresql, typescript
Ambito
database

Direzione di ricerca

Inizia dal percorso VFS del filesystem Node usato da new PGlite(dataDir), quindi traccia il ciclo di vita dell'apertura, della chiusura e dell'uscita del processo. Riproduci il caso con due processi e verifica che un lock attivo identifichi il proprio processo, che un lock obsoleto venga rilevato e che il lock venga rilasciato senza corrompere la directory dei dati.

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

Descrizione

New feature requestPostgres refuses to start on a data directory another postmaster holds (postmaster.pid). PGlite on the Node filesystem VFS takes no such lock, so two processes opening the same dataDir both succeed, both write, and the directory ends up unreadable. The next open then dies with RuntimeError: Aborted() (the symptom in #884) or a checkpoint panic (#327), with nothing pointing at the cause.

The two openers in our case were ordinary: a dev server running with --hot, and a script that imported the same database module. A rm -rf of the data directory under a running server ends the same way.

Describe the solution you'd like

new PGlite(dataDir) on the Node filesystem writes a lock file in the directory holding its process id, and a second open throws when that process is alive, naming it. A stale file (process gone) is taken over. Released on close and on process exit. Same idea as postmaster.pid, scoped to what PGlite can know in one process.

Describe alternatives you've considered

  • We wrote this lock in the application (pid file beside the directory, process.kill(pid, 0) for liveness). It works, but every PGlite user with more than one process hits the same corruption and rediscovers it from Aborted().
  • A documentation note. It does not stop the second open.

Additional context

The lock request in #85 is for the IndexedDB VFS. This one is the Node filesystem VFS, where the second opener is another OS process.

Lingua principale
TypeScript
Stelle
16.3k
Fork
474
Merge medio
23h 53m
PR unite (30g)
53

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 electric-sql/pglite

Tutte le issue di electric-sql/pglite

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.