Bug #80260 [Opn]: PDO::inTransaction() returns false when inside transaction

Edit report at https://bugs.php.net/bug.php?id=80260&edit=1 ID: 80260 Updated by: nikic@php.net Reported by: corey dot taylor dot fl at gmail dot com Summary: PDO::inTransaction() returns false when inside transaction Status: Open Type: Bug Package: PDO MySQL PHP Version: 8.0.0RC2 Block user comment: N Private report: N New Comment: I've opened a PR that should fix your usage (if inTransaction commit), but may break other usages (unconditional commit/rollBack despite an implicit commit). As such, I'm not completely sure whether this will go forward. Previous Comments: ------------------------------------------------------------------------ [2020-10-20 10:03:48] corey dot taylor dot fl at gmail dot com I see now that the implicit commit closes the transaction on the server so there is no valid "commit" after the DDL statement. I thought it just committed the changes and left the transaction open where a commit was expected. We haven't had to inspect the mysql transaction support recently. If the issue is entirely on the PDO side, then we can handle inTransaction() returning false (correctly) as long as beginTransaction() doesn't throw the error. ------------------------------------------------------------------------ [2020-10-20 09:48:20] nikic@php.net The following pull request has been associated: Patch Name: Fix inconsistency in PDO transaction state On GitHub: https://github.com/php/php-src/pull/6355 Patch: https://github.com/php/php-src/pull/6355.patch ------------------------------------------------------------------------ [2020-10-20 08:54:55] corey dot taylor dot fl at gmail dot com If this turns out to be *entirely* due to DDL transactions - are you saying that PDO::inTransaction() won't support them? ------------------------------------------------------------------------ [2020-10-20 08:50:46] corey dot taylor dot fl at gmail dot com I spent some time trying to narrow down the scenarios as you mentioned. It's difficult when it happens in a complex test harness. Yes, the initial failure were showing up from a DDL transaction. I thought the same thing here that maybe there was a sync issue. However, the mysql instance clearly thinks the transaction is ok since it's waiting on a commit. After working on moving the DDL that fails out of the transactions and only putting inserts into the transaction, that initial failure stopped, but the same error occurs further down our test suite. It is possible there is a completely unrelated transaction with DDL in it before this failure, but so far I haven't found it. The test suite really only runs DDL in one place. We don't mix manual and PDO transactions. Everything be through beginTransaction, commit and rollback. ------------------------------------------------------------------------ [2020-10-20 08:44:02] nikic@php.net Okay, next guess: You are running a DDL query inside a transaction, which (for MySQL) will cause an automatic commit, even outside autocommit mode (https://dev.mysql.com/doc/refman/8.0/en/implicit-commit.html). That means inTransaction() reports false, but PDO still thinks its inside a transaction. Does that sound plausible? ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=80260 -- Edit this bug report at https://bugs.php.net/bug.php?id=80260&edit=1