[Bug 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 Kevin Israel (PleaseStand) changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|--- |FIXED Target Milestone|--- |1.23.0 release --- Comment #14 from Kevin Israel (PleaseStand) --- (In reply to Kevin Israel (PleaseStand) from comment #13) > The above change added detection for most versions of MariaDB, though I > still have to fix detection of Percona Server. Decided not to do that, instead removing the broken check in f67543df0675. -- 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 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 Kevin Israel (PleaseStand) changed: What|Removed |Added Status|PATCH_TO_REVIEW |ASSIGNED CC||pleasest...@live.com Assignee|wikibugs-l@lists.wikimedia. |pleasest...@live.com |org | --- Comment #13 from Kevin Israel (PleaseStand) --- The above change added detection for most versions of MariaDB, though I still have to fix detection of Percona Server. -- 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 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 --- Comment #12 from Gerrit Notification Bot --- Change 106489 merged by jenkins-bot: WebInstaller: Don't call getSoftwareLink() https://gerrit.wikimedia.org/r/106489 -- 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 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 --- Comment #11 from Gerrit Notification Bot --- Change 106490 abandoned by PleaseStand: Clean up config-support-* messages Reason: Squashed into I6dbc9d32, with message keys changed. https://gerrit.wikimedia.org/r/106490 -- 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 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 --- Comment #10 from Gerrit Notification Bot --- Change 106489 had a related patch set uploaded by PleaseStand: WebInstaller: Don't call getSoftwareLink() https://gerrit.wikimedia.org/r/106489 -- 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 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 --- Comment #9 from Gerrit Notification Bot --- Change 106490 had a related patch set uploaded by PleaseStand: Clean up config-support-* messages https://gerrit.wikimedia.org/r/106490 -- 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 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 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 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 Daniel Friesen changed: What|Removed |Added CC||mediawiki-bugs@nadir-seen-f ||ire.com --- Comment #8 from Daniel Friesen --- Yeah it's not just MariaDB, it's also Percona Server (Drizzle doesn't support mysql's communication protocol so it's not relevant). The return of VERSION() appears to at least hint what the server type is. We might just want to try and extract which server type it is out of that then look up a hardcoded list of MySQL derived server types and their urls. Creating a subclass for MariaDB doesn't really sound like a great idea. That could make things turn into a mess if we decide to slowly ease into migrating from the mysql_* functions to MySQLi or PDO by creating something like a Database_MySQLi subclass. -- 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 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 --- Comment #7 from Sam Reed (reedy) --- https://en.wikipedia.org/wiki/MySQL#Related_projects -- 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 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 --- Comment #6 from Bawolff (Brian Wolff) --- Probably want to change config-type-mysql msg in installer to be "MySql or MariaDB" -- 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 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 Tim Starling changed: What|Removed |Added CC||tstarl...@wikimedia.org --- Comment #5 from Tim Starling --- Database*::getSoftwareLink() is already being called non-statically from both WebInstallerPage and SpecialVersion. So it would be about 5 minutes' work to change the method declarations to be non-static and to fix DatabaseMysql::getSoftwareLink() to return something appropriate. In the WebInstallerPage case, the object is not connected to an actual server, which is not a problem, it's just an extra line of code to check for that. -- 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 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 Alex Monk (Krenair) changed: What|Removed |Added CC||kren...@gmail.com --- Comment #4 from Alex Monk (Krenair) --- (In reply to comment #3) > I wonder if we actually really need to do anything further for this. From > mediawiki.org just now: > > MySQL5.5.30-MariaDB-mariadb1~precise-log Well it says 'MySQL' and the link goes to http://www.mysql.com/. So while we don't really need to change that, it seems appropriate considering it's not really MySQL, but a fork. -- 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 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 Sam Reed (reedy) changed: What|Removed |Added Priority|Low |Lowest --- Comment #3 from Sam Reed (reedy) --- I wonder if we actually really need to do anything further for this. From mediawiki.org just now: MySQL5.5.30-MariaDB-mariadb1~precise-log -- 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 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 Sumana Harihareswara changed: What|Removed |Added CC||suma...@panix.com --- Comment #2 from Sumana Harihareswara 2011-11-02 22:22:21 UTC --- The issue here is that MediaWiki thinks that the database is MySQL "because we're using mysql_* functions ...we assumed DatabaseMysql would only ever refer to mysql Presumably we'd like to have some way in php to differentiate the two." I'm reaching out to the MariaDB community to see whether they care about this issue at all and would want to put any time into fixing it. IRC conversation for context: <^demon> I dunno what MariaDB would need to do. sumanah: that looks like an issue on the MariaDB side? <^demon> Presumably we'd like to have some way in php to differentiate the two. the only way to fix that is to contribute the patch to them blobaugh: I mean, MariaDB people would donate a bit of time to fix it in MediaWiki. <^demon> Again... <^demon> I'm not sure what MariaDB needs to do? <^demon> We assume it's mysql because we're using mysql_* functions ^demon: it looks like just change the link on the Special:Version page getSoftwareLink() currently returns a static string (DatabaseMysql class) <^demon> Platonides: Right. Because we assumed DatabaseMysql would only ever refer to mysql. the work is to somewhow differenciate between the two so to fix it a new class would need to be made? DatabaseMariaDb? <^demon> Platonides: But other than a software link, do we need to differentiate at all? <^demon> ie: Would the subclass be anything other than overriding getSoftwareLink()? Not in the code, yet make mysql family servers add a command 'SELECT UPSTREAM_LINK();' :) I don't think so ^demon: afaik you are correct if we switched to it, perhaps we would begin optimizing some edge cases I thought MariaDB was meant to be super transparent with regards to working MySQL code? but it's similar enough to mysql that I don't think we would make any change in the class -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- 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 28281] Differentiate between MySQL and MySQL forks (ie MariaDB)
https://bugzilla.wikimedia.org/show_bug.cgi?id=28281 Mark A. Hershberger changed: What|Removed |Added Priority|Normal |Low CC||m...@everybody.org --- Comment #1 from Mark A. Hershberger 2011-03-29 23:59:49 UTC --- I don't see any reason to really "push" this since it isn't MW's responsibility to promote MySQL. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- 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
