[Bug 41170] Transaction warnings during update.php
https://bugzilla.wikimedia.org/show_bug.cgi?id=41170 Andre Klapper aklap...@wikimedia.org changed: What|Removed |Added Keywords|patch-in-gerrit | -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 41170] Transaction warnings during update.php
https://bugzilla.wikimedia.org/show_bug.cgi?id=41170 Nemo federicol...@tiscali.it changed: What|Removed |Added See Also||https://bugzilla.wikimedia. ||org/show_bug.cgi?id=41394 -- You are receiving this mail because: You are on the CC list for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 41170] Transaction warnings during update.php
https://bugzilla.wikimedia.org/show_bug.cgi?id=41170 denny vrandecic denny.vrande...@wikimedia.de changed: What|Removed |Added Status|RESOLVED|VERIFIED -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 41170] Transaction warnings during update.php
https://bugzilla.wikimedia.org/show_bug.cgi?id=41170 denny vrandecic denny.vrande...@wikimedia.de changed: What|Removed |Added Status|NEW |RESOLVED CC||denny.vrandecic@wikimedia.d ||e Resolution||FIXED -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 41170] Transaction warnings during update.php
https://bugzilla.wikimedia.org/show_bug.cgi?id=41170 Daniel Kinzler daniel.kinz...@wikimedia.de changed: What|Removed |Added CC||wikibugs-l@lists.wikimedia. ||org, ||wikidata-bugs@lists.wikimed ||ia.org Component|Database|WikidataRepo Version|1.21-git|unspecified AssignedTo|wikibugs-l@lists.wikimedia. |wikidata-bugs@lists.wikimed |org |ia.org Product|MediaWiki |MediaWiki extensions --- Comment #1 from Daniel Kinzler daniel.kinz...@wikimedia.de 2012-10-18 13:38:05 UTC --- DatabaseUpdater::doUpdates wraps all updates in one big transaction (this was introduces by Iebb6855e, apparently to fix some issue with PG). If some DB update registered by an extension tries to open a transaction, that will cause these warnings (if you have $wgDevelopmentWarnings enabled). Wikibase\Utils::insertDefaultSites() does do that. There's two possible solutions: 1) remove the global spanning transaction. That seems to be prudent in the absence of support for nested transactions. But it may break whatever problem was fixed by Iebb6855e. 2) detect open transactions in Utils::insertDefaultSites(), and only open a transaction if there isn't one already. Also, I'm not sure why Utils::insertDefaultSites() needs a transaction at all. Consistency is not critical here. Is it a performance thing? -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 41170] Transaction warnings during update.php
https://bugzilla.wikimedia.org/show_bug.cgi?id=41170 Daniel Kinzler daniel.kinz...@wikimedia.de changed: What|Removed |Added Keywords||patch-in-gerrit --- Comment #2 from Daniel Kinzler daniel.kinz...@wikimedia.de 2012-10-18 13:50:41 UTC --- patch submitted as I2348c28b -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l