É totalmente desnecessário, pois cada instrução SQL é executada atomicamente, ou seja. como se já estivesse rodando em sua própria transação. Na verdade, a abertura de transações desnecessárias pode levar a um aumento de bloqueios, até mesmo de deadlocks. Esquecer de combinar COMMITs com BEGINs pode deixar uma transação aberta enquanto a conexão com o banco de dados estiver aberta e interferir em outras transações na mesma conexão.

Essa codificação quase certamente significa que quem escreveu o código não era muito experiente em programação de banco de dados e é um cheiro certo de que pode haver outros problemas também.