[Bug 41170] Transaction warnings during update.php
https://bugzilla.wikimedia.org/show_bug.cgi?id=41170 Andre Klapper 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 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 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 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 changed: What|Removed |Added Keywords||patch-in-gerrit --- Comment #2 from Daniel Kinzler 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
[Bug 41170] Transaction warnings during update.php
https://bugzilla.wikimedia.org/show_bug.cgi?id=41170 Daniel Kinzler 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 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