Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dev/core#392 Fix FTS INNDOB indexer query to work in MySQL 8 as well … #16202

Merged
merged 1 commit into from
Jan 6, 2020

Conversation

seamuslee001
Copy link
Contributor

…as previous MySQL versions

Overview

In MySQL 8 the INNODB tables in the information schema have had the _SYS_ removed from them as per https://dev.mysql.com/doc/refman/8.0/en/mysql-nutshell.html#mysql-nutshell-removals

Before

CRM_Utils_QueryFormatterTest fails on MySQL 8

After

CRM_Utils_QueryFormatterTest passes on MySQL 8

ping @totten @eileenmcnaughton @monishdeb @JoeMurray

@civibot
Copy link

civibot bot commented Jan 3, 2020

(Standard links)

@civibot civibot bot added the master label Jan 3, 2020
@seamuslee001
Copy link
Contributor Author

Jenkins re test this please

@monishdeb
Copy link
Member

Rightful fix for mysql8 and doesn't break current >Mysql8 instance, confirmed by passed test build. Merging it now

@monishdeb monishdeb merged commit c143d7c into civicrm:master Jan 6, 2020
@eileenmcnaughton eileenmcnaughton deleted the mysql_8_information_schema branch January 6, 2020 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants