User "Raymond" changed the status of MediaWiki.r90643. Old Status: new New Status: fixme
User "Raymond" also posted a comment on MediaWiki.r90643. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/90643#c18594 Commit summary: General database-related code cleanup: * Merged the 4 simulated implementations of Database*::replace(). I took diffs, they were nearly identical. I made one based on the IBM DB2 version, since it used insert() which looked like a nice touch. * Provided the non-simulated implementation of Database*::replace() via a protected member function, and made DatabaseMysql::replace() and DatabaseSqlite::replace() into a wrapper for it. * Moved the MySQL-specific functionality from masterPosWait(), getSlavePos() and getMasterPos() from DatabaseBase to DatabaseMysql. * Renamed getStatus() to getMysqlStatus() and moved it to DatabaseMysql. Removed "unimplemented" errors from two other subclasses. Really there's no way another DBMS could or should implement this function. * Split the LoadMonitor class into a no-op class and a MySQL-specific class, to avoid fatal errors due to the getMysqlStatus() call if other DBMSs tried to use LoadBalancer with multiple servers. Of course there are lots of other reasons it won't work, I'm just fixing the architecture issue here. And while I have a replicated test setup handy: * On MySQL 4.1.9 and later, use SHOW SLAVE STATUS to get the lag instead of SHOW PROCESSLIST. This has the advantage of reading zero when there are no events for a while. Comment: Error on Translatewiki, running MySQL 5.1.41-3ubuntu12.10-log : Es ist ein Datenbankfehler aufgetreten. Der Grund kann ein Programmierfehler sein. Die letzte Datenbankabfrage lautete: <blockquote><tt>(SQL-Abfrage versteckt)</tt></blockquote> aus der Funktion „<tt>DatabaseMysql::getLagFromSlaveStatus</tt>“. Die Datenbank meldete den Fehler „<tt>1227: Access denied; you need the SUPER,REPLICATION CLIENT privilege for this operation (localhost)</tt>“. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
