[Bug 63145] Referential integrity issues on commons on pages created by API

2014-04-06 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=63145

--- Comment #8 from Gerrit Notification Bot  ---
Change 124137 merged by jenkins-bot:
Make doEditContent call $dbw->rollback() if exception happens

https://gerrit.wikimedia.org/r/124137

-- 
You are receiving this mail because:
You are the assignee for the bug.
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 63145] Referential integrity issues on commons on pages created by API

2014-04-05 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=63145

Bawolff (Brian Wolff)  changed:

   What|Removed |Added

 Status|PATCH_TO_REVIEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #7 from Bawolff (Brian Wolff)  ---
(In reply to Gerrit Notification Bot from comment #6)
> Change 124137 had a related patch set uploaded by Brian Wolff:
> Make doEditContent call $dbw->rollback() if exception happens
> 
> https://gerrit.wikimedia.org/r/124137

This patch is related to making certain things more robust, but ultimately this
bug is already fixed, so resetting to fixed.

-- 
You are receiving this mail because:
You are the assignee for the bug.
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 63145] Referential integrity issues on commons on pages created by API

2014-04-05 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=63145

Gerrit Notification Bot  changed:

   What|Removed |Added

 Status|RESOLVED|PATCH_TO_REVIEW
 Resolution|FIXED   |---

-- 
You are receiving this mail because:
You are the assignee for the bug.
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 63145] Referential integrity issues on commons on pages created by API

2014-04-05 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=63145

--- Comment #6 from Gerrit Notification Bot  ---
Change 124137 had a related patch set uploaded by Brian Wolff:
Make doEditContent call $dbw->rollback() if exception happens

https://gerrit.wikimedia.org/r/124137

-- 
You are receiving this mail because:
You are the assignee for the bug.
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 63145] Referential integrity issues on commons on pages created by API

2014-03-31 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=63145

Brad Jorsch  changed:

   What|Removed |Added

 Status|PATCH_TO_REVIEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Brad Jorsch  ---
This should be deployed to WMF wikis with 1.23wmf21, see
https://www.mediawiki.org/wiki/MediaWiki_1.23/Roadmap for the schedule.

-- 
You are receiving this mail because:
You are the assignee for the bug.
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 63145] Referential integrity issues on commons on pages created by API

2014-03-31 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=63145

--- Comment #4 from Gerrit Notification Bot  ---
Change 121402 merged by jenkins-bot:
Improve handling of uncommitted DB txns with "uncaught" exceptions

https://gerrit.wikimedia.org/r/121402

-- 
You are receiving this mail because:
You are the assignee for the bug.
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 63145] Referential integrity issues on commons on pages created by API

2014-03-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=63145

--- Comment #3 from Gerrit Notification Bot  ---
Change 121402 had a related patch set uploaded by Anomie:
Improve handling of uncommitted DB txns with "uncaught" exceptions

https://gerrit.wikimedia.org/r/121402

-- 
You are receiving this mail because:
You are the assignee for the bug.
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 63145] Referential integrity issues on commons on pages created by API

2014-03-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=63145

Gerrit Notification Bot  changed:

   What|Removed |Added

 Status|NEW |PATCH_TO_REVIEW

-- 
You are receiving this mail because:
You are the assignee for the bug.
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 63145] Referential integrity issues on commons on pages created by API

2014-03-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=63145

--- Comment #2 from Brad Jorsch  ---
(In reply to Bawolff (Brian Wolff) from comment #0)
> *On index.php, transaction will be thrown away (implicitly anyways,
> basically the shutdown method isn't called)

That is probably what's behind bug 56269.

> Happens regardless of if there is an uncaught exception, triggering a
> $dbw->commit even if exception was thrown in middle of transaction.

Yeah, that's no good.

Of course, in both this bug and bug 56269 the real solution is that whatever is
opening these transactions should catch, rollback, and rethrow. But uncaught
exceptions happen, so the last-resort catch should DTRT and both rollback and
log the stack trace of the problematic exception.

-- 
You are receiving this mail because:
You are the assignee for the bug.
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 63145] Referential integrity issues on commons on pages created by API

2014-03-26 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=63145

Tomasz W. Kozlowski  changed:

   What|Removed |Added

   Priority|Unprioritized   |Normal

--- Comment #1 from Tomasz W. Kozlowski  ---
This also appears to be happening on wikis other than Commons, though on a much
more limited scale than on Commons (one page on frwiki, for instance).

-- 
You are receiving this mail because:
You are the assignee for the bug.
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