Closed (fixed)
Project:
Provision
Version:
7.x-3.x-dev
Component:
DB Service
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Jul 2020 at 15:43 UTC
Updated:
11 Dec 2020 at 13:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
sseidel commentedComment #3
sseidel commentedComment #4
martin_qI have not tested this but I have looked over the patch. It seems sane to me. A couple of probably very minor suggestions, which may not even be anything anyone cares about:
Does it need an 'identical' comparison or would
if ($this->server->utf8mb4_is_supported)be sufficient?The original string doesn't contain
\nbut instead literal ends-of-lines, so does this patch introduce unnecessary inconsistency?Comment #6
helmo commentedThanks, committed ... with @martin_q's remarks.
PS: an alternative/workaround is to add that to the global mysql config in the [mysqldump] section.