Last modified: 2014-10-07 19:11:30 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T68492, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 66492 - Add mathoid database table
Add mathoid database table
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
General/Unknown (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: schema-change
Depends on:
Blocks: 49188
  Show dependency treegraph
 
Reported: 2014-06-11 19:11 UTC by physikerwelt
Modified: 2014-10-07 19:11 UTC (History)
8 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description physikerwelt 2014-06-11 19:11:54 UTC
Short description:
Add the mathoid table that stores MathML and SVG output for texvc markup.
It is seperated from the old math table called math.
Which wikis are affected:
all
Which tables:
mathoid
What is the change to those tables:
new table
Links to gerrit changes and/or other related bug reports:
https://gerrit.wikimedia.org/r/#/c/135521/
https://bugzilla.wikimedia.org/show_bug.cgi?id=65793
Comment 1 physikerwelt 2014-06-11 19:17:49 UTC
Report created via template from https://wikitech.wikimedia.org/wiki/Schema_changes
To be honest I don't understand why this table has to be created manually.
Comment 2 Greg Grossmeier 2014-06-11 20:24:37 UTC
This bug is a Wikimedia-specific bug. There is already the Math extension specific bug at bug 65793.

This probably won't be done before next week so if there are backwards-incompatible changes committed in the Math extension, they should be reverted now, ie: before 10:00 UTC on Thursday the 12th.
Comment 3 Gerrit Notification Bot 2014-06-11 21:04:01 UTC
Change 138993 had a related patch set uploaded by Physikerwelt:
Disable MathML rendering option

https://gerrit.wikimedia.org/r/138993
Comment 4 Gerrit Notification Bot 2014-06-12 06:06:49 UTC
Change 139068 had a related patch set uploaded by Physikerwelt:
Disable MathML rendering option

https://gerrit.wikimedia.org/r/139068
Comment 5 Gerrit Notification Bot 2014-06-12 06:08:21 UTC
Change 138993 abandoned by Physikerwelt:
Disable MathML rendering option

Reason:
moved Icf0c462b57af6e0fc0bc80994d07b004f2829f56

https://gerrit.wikimedia.org/r/138993
Comment 6 physikerwelt 2014-06-12 14:36:06 UTC
OK. The change was not merged and no problem was reported.
Maybe there is no problem.
If there is a problem the change

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

will disable the feature that potentially causes a problem due to the missing mathoid table.
Comment 7 physikerwelt 2014-06-12 22:17:26 UTC
OK. Now the new branch has been created and the expected can be produced.
"
Database error
A database query error has occurred. This may indicate a bug in the software.

Function: MathRenderer::readFromDatabase
Error: 1146 Table 'mediawikiwiki.mathoid' doesn't exist (10.64.16.8)
"
From
https://www.mediawiki.org/wiki/User:Physikerwelt
rendered with MathML mode.
Comment 8 Gerrit Notification Bot 2014-06-12 22:45:24 UTC
Change 139068 merged by jenkins-bot:
Disable MathML rendering option

https://gerrit.wikimedia.org/r/139068
Comment 9 Greg Grossmeier 2014-06-12 22:49:07 UTC
This has caused headaches all day. There is a reason we have a process for adding new tables in production. We don't just run install.php in production.

In the future, if you don't follow https://wikitech.wikimedia.org/wiki/Schema_changes we'll have to fully revert the math extension or completely turn it off. Continued recklessness in this area is not sustainable.
Comment 10 MZMcBride 2014-06-13 01:26:46 UTC
(In reply to Greg Grossmeier from comment #9)
> This has caused headaches all day. There is a reason we have a process for
> adding new tables in production. We don't just run install.php in production.
> 
> In the future, if you don't follow
> https://wikitech.wikimedia.org/wiki/Schema_changes we'll have to fully
> revert the math extension or completely turn it off. Continued recklessness
> in this area is not sustainable.

(In reply to Ori Livneh from bug 65793 comment #15)
> I reverted to 1bb3bfa3b5656 because the exception log was flooded with
> errors related to the math extension. I also revoked +2 rights on
> extension/Math from Physikerwelt and Frédéric Wang until Greg has had a
> chance to review exactly what happened.

I may be turned around and confused about the order of events here and I'll be interested to read Greg's formal review.

That said, I get the impression that both of these quoted comments are placing the responsibility of deploying code and maintaining Wikimedia Foundation servers on the shoulders of two particular extension authors. This seems a bit... wrong and backward to me.

It looks to me like one of these two volunteers was kind enough to (promptly!) provide a Wikimedia-specific patch after being asked to, which he was under no obligation to do, and that patch then didn't get merged and deployed until some 16 hours later. This issue was causing problems "all day," eh?

If the Wikimedia Foundation isn't reviewing extension code before merging it into the release branches and shipping it to the servers, that sounds like a pretty major bug to me... but I'm not sure why people are calling the volunteer behavior here reckless and revoking access when it's the non-volunteer behavior here that seems to need investigation.
Comment 11 physikerwelt 2014-06-13 04:25:45 UTC
I feel sorry for not knowing that there is a page
https://wikitech.wikimedia.org/wiki/Schema_changes .
For the future it might help to add a note to
https://www.mediawiki.org/wiki/Manual:Hooks/LoadExtensionSchemaUpdates
and
https://www.mediawiki.org/wiki/Manual:Developing_extensions#Adding_database_tables
.
Comment 12 Brad Jorsch 2014-06-13 14:44:35 UTC
(In reply to MZMcBride from comment #10)
> That said, I get the impression that both of these quoted comments are
> placing the responsibility of deploying code and maintaining Wikimedia
> Foundation servers on the shoulders of two particular extension authors.
> This seems a bit... wrong and backward to me.

Anyone, volunteer or staff, with +2 access on MediaWiki core, WMF-deployed extensions, or other repositories that affect the WMF sites is expected to take part in ensuring that that code doesn't negatively affect the WMF sites. This is a requirement of having +2 access to those repositories, and is stated on https://www.mediawiki.org/wiki/Gerrit/%2B2

If someone does not want this responsibility, then they shouldn't have +2. We'll still welcome their contributions, just like the wiki communities welcome contributions from people who aren't admins or 'crats.

It would certainly be doable for only WMF staff to be allowed +2 to these repositories as you're saying should be the case, although it would slow down development in some areas. But why *shouldn't* we accept help in this from skilled volunteers as we do in so many other areas?
Comment 13 Sam Reed (reedy) 2014-06-13 14:55:22 UTC
bug 66587 suggests "Leave it as TeX (for text browsers)" doesn't work either
Comment 14 Sam Reed (reedy) 2014-06-13 14:56:29 UTC
Is there any reason the table can't be added now?
Comment 15 Gerrit Notification Bot 2014-06-13 14:57:46 UTC
Change 139381 had a related patch set uploaded by Reedy:
Remove MW_MATH_SOURCE as errors without mathoid table

https://gerrit.wikimedia.org/r/139381
Comment 16 Gerrit Notification Bot 2014-06-13 14:59:14 UTC
Change 139381 merged by jenkins-bot:
Remove MW_MATH_SOURCE as errors without mathoid table

https://gerrit.wikimedia.org/r/139381
Comment 17 physikerwelt 2014-06-13 15:24:56 UTC
Can anyone comment on this comment I made on the other bug-ticket:
"
https://gerrit.wikimedia.org/r/#/c/138023/
I could image that the wmf8 code produces database write operation that are not required.
"
I think that could be relevant.
Comment 18 Sam Reed (reedy) 2014-06-13 16:36:52 UTC
Table added
Comment 19 Gerrit Notification Bot 2014-06-13 16:37:02 UTC
Change 139421 had a related patch set uploaded by Reedy:
Re-enable all Math modes

https://gerrit.wikimedia.org/r/139421
Comment 20 Greg Grossmeier 2014-06-13 22:47:14 UTC
The table is still created, even though Gerrit Notification Bot reopened this :)
Comment 21 Gerrit Notification Bot 2014-06-30 10:55:31 UTC
Change 135522 had a related patch set uploaded by Physikerwelt:
Add table for new math renderer Mathoid

https://gerrit.wikimedia.org/r/135522
Comment 22 Gerrit Notification Bot 2014-08-12 14:49:10 UTC
Change 135522 merged by jenkins-bot:
Add table for new math renderer Mathoid

https://gerrit.wikimedia.org/r/135522
Comment 23 Gerrit Notification Bot 2014-08-12 15:05:34 UTC
Change 153610 had a related patch set uploaded by Physikerwelt:
Re-enable all Math modes on beta

https://gerrit.wikimedia.org/r/153610
Comment 24 Gerrit Notification Bot 2014-10-07 19:11:30 UTC
Change 139421 merged by jenkins-bot:
Re-enable all Math modes

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

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links