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

InnoDB compliance. Is FULLTEXT index really needed? #42

Closed
liluxdev opened this issue Oct 8, 2013 · 3 comments
Closed

InnoDB compliance. Is FULLTEXT index really needed? #42

liluxdev opened this issue Oct 8, 2013 · 3 comments

Comments

@liluxdev
Copy link

liluxdev commented Oct 8, 2013

Joomla 3.1 now enforces tables to InnoDB (when installed from scratch, not in migration)

See:

https://github.com/joomla/joomla-cms/blob/master/installation/sql/mysql/joomla.sql#L85

I think that the time when MyISAM was more performat and available is gone, so consider to permit InnoDB also in K2...

But to permit InnoDB you have to remove the FULLKEY indexes as they are not supported by the engine and declaring them makes impossible an InnoDB conversion:

https://github.com/joomlaworks/k2/blob/master/administrator/components/com_k2/install.mysql.sql#L121

Are them really used? Maybe in the search plugin but now Joomla as a new search engine too, com_finder, which does optimizations, indexing and taxonomy for us.

@kavadas
Copy link
Contributor

kavadas commented Oct 9, 2013

Hi. This what we are going to do for the next major release of K2. Right now i think there is no point changing the database structure.

@dgrammatiko
Copy link

Lefteris is there a roadmap somewhere? This next version (i guess is 3?) when is it due? I have some code that I want to bring to the project

@fevangelou
Copy link
Member

K2 version 3 is still under development. The code will be published here on Github near the end of November 2013. A blog post will be published on getk2.org providing more details within this month.

liluxdev added a commit to liluxdev/k2 that referenced this issue Oct 16, 2013
one query with GROUP BY is enough to get this info
liluxdev added a commit to liluxdev/k2 that referenced this issue Oct 16, 2013
one query with GROUP BY is enough to get this info
@kavadas kavadas closed this as completed Oct 30, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants