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

menu_tree_check_access() adds 1141 database queries #2825

Open
jenlampton opened this issue Sep 3, 2017 · 0 comments
Open

menu_tree_check_access() adds 1141 database queries #2825

jenlampton opened this issue Sep 3, 2017 · 0 comments

Comments

@jenlampton
Copy link
Member

jenlampton commented Sep 3, 2017

Describe your issue or idea

When I upgraded a site from Drupal 6 to Backdrop CMS, I noticed front-end pages getting sluggish, while the back end stayed quick and snappy. When removing the menu block, the front-end of my site was also quick and snappy.

After much further digging (thanks the devel query log!) I have tracked the problem down to the function menu_tree_check_access() which adds an additional 1141 database queries to my page.

It's true - my menu tree is quite large. I still think there's something we could do to speed this up. I don't see why these queries need to be run uniquely for every visitor on every page of my site -
when the links are the same on every page.

@jenlampton jenlampton changed the title menu_tree_check_access() adds 1141 database queries to my page menu_tree_check_access() adds 1141 database queries Sep 3, 2017
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

1 participant