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

Increase performance of Extensions table? #5101

Closed
javigomez opened this issue Nov 14, 2014 · 4 comments · Fixed by #5120
Closed

Increase performance of Extensions table? #5101

javigomez opened this issue Nov 14, 2014 · 4 comments · Fixed by #5120

Comments

@javigomez
Copy link
Contributor

Hi, I just saw this in my debug:

screen shot 2014-11-14 at 13 15 55

I checked and the Indexes in this table are:

screen shot 2014-11-14 at 13 16 34

I'm not skilled in query performance but I was wondering if we can add some Indexes to "enabled", "type", "state"...

What do you think?

cc: @beat @gunjanpatel, @nadeeshaan, @alikon, @aschkenasy, @mrunalpittalia, and rest of members of the http://docs.joomla.org/SQL_Optimisation_Working_Group

@aschkenasy
Copy link

State can't be indexed (index will not help.) Since it's a range lookup.
Type is already indexed as first element of the extension index. I fear that the range lookup of enabled is preventing the query optimizer to do use the index.

On November 14, 2014 7:18:52 AM EST, javier gomez notifications@github.com wrote:

Hi, I just saw this in my debug:

screen shot 2014-11-14 at 13 15 55

I checked and the Indexes in this table are:

screen shot 2014-11-14 at 13 16 34

I'm not skilled in query performance but I was wondering if we can add
some Indexes to "enabled", "type", "state"...

What do you think?

cc: @beat @gunjanpatel, @nadeeshaan, @alikon, @aschkenasy,
@mrunalpittalia, and rest of members of the
http://docs.joomla.org/SQL_Optimisation_Working_Group


Reply to this email directly or view it on GitHub:
#5101

Sent from my mobile device. Please excuse my brevity.

@Bakual
Copy link
Contributor

Bakual commented Nov 14, 2014

It should work if the query is changed to AND state IN (0,1), right? Then state could be indexed and it could be used.

However I wonder what the purpose of that column is. In my test database this value is always 0. We don't use a state there to my knowledge. We use the enabled column instead.

@aschkenasy
Copy link

State in is perfect.
Likewise any finite option should be
IN(1,2,3,...)

On November 14, 2014 10:00:57 AM EST, Thomas Hunziker notifications@github.com wrote:

It should work if the query is changed to AND state IN (0,1), right?
Then state could be indexed and it could be used.

However I wonder what the purpose of that column is. In my test
database this value is always 0. We don't use a state there to my
knowledge. We use the enabled column instead.


Reply to this email directly or view it on GitHub:
#5101 (comment)

Sent from my mobile device. Please excuse my brevity.

@Hackwar
Copy link
Member

Hackwar commented Nov 16, 2014

See #5120

Bakual pushed a commit that referenced this issue Nov 21, 2014
Improving query to use indexes better. Fixes #5101
@Bakual Bakual added this to the Joomla! 3.4.0 milestone Nov 21, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants