Come evitare le condizioni di gara quando si utilizza il metodo find_or_create di DBIx :: Class :: ResultSet?

Question

Dalla documentazione per find_or_create:

Nota: A causa find_or_create() legge dal database e poi, eventualmente, inserti in base al risultato, questo metodo è soggetto a una gara condizione. Un altro processo potrebbe creare un record nella tabella dopo la ricerca è stata completata e prima dell'avvio della creazione. Per evitare questo problema, utilizzare find_or_create() all'interno di una transazione.

E 'sufficiente utilizzare solo find_or_create() all'interno di una transazione in PostgreSQL?

Answer 1

No, la documentazione non è corretta. L'utilizzo di una transazione da solo non fa non evitare questo problema. Garantisce solo il rollback dell'intera transazione se si verifica un'eccezione, in modo che nessuno stato incoerente venga mantenuto nel database.

Per evitano questo problema è necessario bloccare la tabella - all'interno di una transazione, in quanto tutti i blocchi vengono rilasciati al termine di una transazione. Qualcosa di simile:

BEGIN; 
LOCK TABLE mytbl IN SHARE MODE; 

-- do your find_or_create here 

COMMIT; 

Ma non è una cura magica per tutto. Può diventare un problema di prestazioni e potrebbero esserci problemi deadlock (transazioni simultanee che provano a bloccare le risorse che l'altro ha già bloccato). PostgreSQL rileverà tale condizione e annullerà tutte le transazioni concorrenti tranne una. Devi essere pronto a riprovare l'operazione in caso di fallimento.

The PostgreSQL manual about locks.

Se non si dispone di un sacco di concorrenza si potrebbe anche semplicemente ignorare il problema. La fascia oraria è molto piccola, quindi accade davvero molto raramente. Se riscontri l'errore di violazione della chiave duplicata, che non danneggerà, avrai coperto anche questo.

Answer 2

Questa implementazione find_or_create dovrebbe impedire la condizione di competizione, descritto nel PO:

eval { 
    $row = $self->model->create({ ... }); 
} 
if($@ && $@ =~ /duplicate/i) { 
    $row = $self->model->find({ ... }); 
} 

Si riduce anche find_or_create() a una singola query nel migliore dei casi.