Sep 13 2019
Sep 8 2019
Sep 7 2019
Now that we are on RDS I can confirm that MySQL also has ONLY_FULL_GROUP_BY enabled by default.
Sep 4 2019
That actually using MySQL official Docker image.
Sep 3 2019
There are only 23 occurrences of the string "GROUP BY" in the codebase, and, from inspection, many obviously do not conflict with ONLY_FULL_GROUP_BY.
Sep 1 2019
Anecdata: locally, using 2 subprocesses went twice as fast (~85s -> ~42s). 4 subprocesses chopped another ~20% of the time off (~42s > ~35s). It stopped getting faster at 4. However, the largest table took 24s, so even if this was completely parallelizable we wouldn't expect it to drop lower than that.
Aug 31 2019
Jun 19 2019
Getting rid of the indirect writes on handle reads would probably be nice eventually, but doesn't directly accomplish anything today.
Jun 18 2019
Jun 5 2019
I applied this change to 20180208.maniphest.02.populate.php:
Apr 15 2019
This remains desirable, but probably won't happen without a stronger motivator since the patches are somewhat involved.
Jan 18 2019
Nothing really actionable here.
Aug 30 2018
Aug 22 2018
- your business faces regulatory/compliance issues which are satisfied by encrypting the connection, even if it provides no direct security benefit.
Mar 18 2018
I don't run my Phabricator instance on Ubuntu but instead on RHEL6 (and with PHP7, which I know isn't officially support)
Mar 13 2018
(I'll follow up in T13102 if there's more to be done here, it's possible that some installs are missing this for reasons other than "Ubuntu + apt get install php5-mysql").
Sep 20 2017
Sep 11 2017
We generally stopped seeing this after moving to InnoDB FULLTEXT, which seems less prone to table crashes than MyISAM FULLTEXT was. See also T12819 for the fate of InnoDB FULLTEXT.
Aug 30 2017
Whoops this is probably my bad :-/
Aug 17 2017
Aug 15 2017
Closing this for lack of feedback, feel free to resurrect it if you get back to it.
Jul 9 2017
May 5 2017
Apr 13 2017
This could definitely be more clear, but I think this is currently expected -- you still need to have valid configuration even when providing additional administrative credentials.
Mar 16 2017
Do this man
Jan 17 2017
Nov 22 2016
I've marked D16909 as fixing this, but it just removes mysql.implementation, and uses the rule "mysqli if available, mysql otherwise".