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

Change MyISAM tables to InnoDB for datastore #2042

Closed
sheldonrampton opened this issue Aug 7, 2017 · 2 comments
Closed

Change MyISAM tables to InnoDB for datastore #2042

sheldonrampton opened this issue Aug 7, 2017 · 2 comments

Comments

@sheldonrampton
Copy link

sheldonrampton commented Aug 7, 2017

DKAN currently uses MyISAM rather than InnoDB when creating datastore tables. This can create some performance issues including slow database cloning between environment on Acquia-hosted websites. The original decision to use MyISAM was motivated by a desire to enable full-text searches of datastore content, but InnoDB supports full-text search in MySQL 5.5 and later, so we should use InnoDB for datastore tables.

A MySQL query for converting MyISAM to InnoDB can be found here:

https://dba.stackexchange.com/questions/50028/mysql-command-querying-all-myisam-database/50030#50030

Acceptance criteria

  • When DKAN creates a new datastore table, that table should be an InnoDB table.
  • When upgrading from earlier releases of DKAN, this fix should include a hook_update function that converts any existing MyISAM tables to InnoDB.
@kimwdavidson
Copy link

Need to decide if we want to do the second acceptance criteria. Or if we just provide documentation for people who want to do this manually.

@erogray
Copy link

erogray commented Mar 31, 2020

We tested with one client who had a lot of datastore tables and it didn't cause significant improvement; closing this as a won't do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants