nekokak/p5-Teng — pull request #126 by GeJ (2014-04-04 03:59:31+00:00)
Title: Allow to use an alternate transaction manager class.

First of all, I'd like to say that DBIx::TransactionManager performs wonderfully under normal workloads. However, while working on a project at $work, we hit a problem where DBIx::TM would not commit a "main" transaction because we had to rollback an "inner" transaction.

I've been working on an alternate transaction manager (soon to be released either as part of DBIx::TM if nekokak-さん agrees or in its own distribution) that handle nested transactions through savepoints and I would like to be able to use it with Teng.

Right now my choices are :
- apply a local patch everytime I deploy/update Teng,
- monkey-patch Teng::txn_manager(), or...
- submit a PR ;)

I have mixed feelings about documenting this feature. On the one hand it might be interesting for people to know about it, on the other hand it would expose 1) Teng's internals and 2) DBIx::TransactionManager's API to the users. I will gladly amend my patch to add documentation if you think it is necessary.

Thank you for considering this patch.