No hay necesidad de transacción al crear historial

alexyslozada / shorturl

Inspired in bit.ly

MIT License

3 stars 2 forks source link

No hay necesidad de transacción al crear historial #5

Closed CarosDrean closed 2 years ago

CarosDrean commented 2 years ago

https://github.com/alexyslozada/shorturl/blob/a003bd1772ede6a9eede1564795a19d8a50637fa/handler/hecho/core/handler.go#L38-L61

Realmente no veo la necesidad que la creación del historial se haga con una transacción, ya que es solo un proceso (si debería seguir en una gorutina), y debería estar en el dominio, no en la infraestructura

alexyslozada commented 2 years ago

Ah, es que dentro del registro del historial hago un incremento a los times del shorturl. Tal vez está confuso de leer. Tienes alguna sugerencia para esto?

alexyslozada commented 2 years ago

Hemos dejado de usar transacciones