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

initial work for alternative mysql engine support (InnoDB, TokuDB, XtraD... #6263

Closed
wants to merge 1 commit into from

Conversation

imreFitos
Copy link

Database setup: add option to choose Database engine type. Only InnoDB right now, although I have tested the latest stable piwik with TokuDB and it had no problems at all.

@mattab
Copy link
Member

mattab commented Sep 24, 2014

Thanks for the pull request @imreFitos

By default there is only Innodb to choose from. To keep things simple maybe you could hide the control from the installer UI until there are at least two choices available?

@mattab mattab added this to the Piwik 2.8.0 milestone Sep 24, 2014
@mattab mattab added the Pull Request WIP Indicates the current pull request is still work in progress and not ready yet for a review. label Sep 24, 2014
@mattab mattab modified the milestones: Short term, Piwik 2.8.0 Oct 2, 2014
@mattab
Copy link
Member

mattab commented Nov 3, 2014

Thank you for this proposed pull request.

Because it was last updated more than one month ago, it is our policy to close pull requests opened for a long time without updates. If you would like to continue work on the pull request, please simply ping us to have it re-opened (after you have pushed a new commit).

We hope you understand this and we look forward to seeing an update from you on this pull request or another one!

Thanks.

@mattab mattab closed this Nov 3, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pull Request WIP Indicates the current pull request is still work in progress and not ready yet for a review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants