Es completamente innecesario ya que cada instrucción SQL se ejecuta atómicamente, es decir. como si ya se estuviera ejecutando en su propia transacción. De hecho, abrir transacciones innecesarias puede llevar a un mayor bloqueo, incluso a puntos muertos. Olvidarse de hacer coincidir COMMIT con BEGIN puede dejar una transacción abierta mientras la conexión a la base de datos esté abierta e interferir con otras transacciones en la misma conexión.
Es casi seguro que dicha codificación signifique que quien escribió el código no tenía mucha experiencia en la programación de bases de datos y es un olor seguro de que también puede haber otros problemas.
