Cómo evitar las condiciones de carrera cuando se usa el método find_or_create de DBIx :: Class :: ResultSet?

Question

De la documentación para find_or_create:

Nota: Debido a find_or_create() lee de la base de datos y luego posiblemente insertos en base al resultado, este método está sujeto a una condición de carrera . Otro proceso podría crear un registro en la tabla después de el hallazgo se ha completado y antes de que la creación haya comenzado. Para evitar este problema, use find_or_create() dentro de una transacción.

¿Es suficiente simplemente usar find_or_create() dentro de una transacción en PostgreSQL?

Answer 1

No, la documentación es incorrecta. El uso de una transacción sola hace que no evite este problema. Solo garantiza que se revierte toda la transacción si se produce una excepción, de modo que no se conservará ningún estado incoherente en la base de datos.

Para evitar este problema debe bloquear la tabla, dentro de una transacción, porque todos los bloqueos se liberan al final de una transacción. Algo como:

BEGIN; 
LOCK TABLE mytbl IN SHARE MODE; 

-- do your find_or_create here 

COMMIT; 

Pero eso no es una cura mágica para todo. Puede convertirse en un problema de rendimiento, y puede haber deadlocks (transacciones simultáneas que intentan bloquear recursos que el otro ya tiene bloqueados). PostgreSQL detectará dicha condición y cancelará todas menos una de las transacciones competidoras. Debe estar preparado para volver a intentar la operación en caso de falla.

The PostgreSQL manual about locks.

Si usted no tiene una gran cantidad de concurrencia es posible que también acaba de ignorar el problema. El intervalo de tiempo es muy pequeño, por lo que muy pocas veces sucede realmente. Si detecta el error de violación de la clave duplicada, que no causará daños, entonces también habrá cubierto esto.

Answer 2

Esta implementación de find_or_create debe impedir que la condición de carrera, que se describe en el PO:

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

También reduce find_or_create() a una sola consulta en el mejor de los casos.