[wp-trac] [WordPress Trac] #22189: Saving navigation menus is slow

WordPress Trac wp-trac at lists.automattic.com
Mon Oct 15 04:18:38 UTC 2012


#22189: Saving navigation menus is slow
--------------------------+------------------
 Reporter:  nacin         |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  3.5
Component:  Menus         |     Version:
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+------------------
Description changed by nacin:

Old description:

> This is a follow-up to a few tickets, such as #17031. I could not find a
> duplicate. It is also tangentially related to #14134, which would be
> menus are affected to max post var settings.
>
> Bottom line: saving navigation menus are slow. I was hoping to see that
> removing hierarchy ancestor cleaning in #11399 made a big improvement,
> but a 90-item flat menu still took 24 seconds and a few ''thousand''
> queries to save. No me gusta.
>
> I started chipping away at this giant list of queries, and came up with a
> number of changes across a number of areas of core to reduce them. The
> page now saves (and renders — it doesn't redirect) with only a few
> hundred queries — it's a lot, but considering a few queries per save, not
> bad — and just a few short seconds on my little MacBook Air that could.
>
> Since these improvements are largely outside the menus API, I will be
> spawning individual tickets for most things.

New description:

 This is a follow-up to a few tickets, such as #17031. I could not find a
 duplicate. It is also tangentially related to #14134, which would be menus
 are affected to max post var settings.

 Bottom line: saving navigation menus are slow. I was hoping to see that
 removing hierarchy ancestor cleaning in #11399 made a big improvement, but
 a 90-item flat menu still took 24 seconds and a few ''thousand'' queries
 to save. No me gusta.

 I started chipping away at this giant list of queries, and came up with a
 number of changes across a number of areas of core to reduce them. The
 page now saves (and renders — it doesn't redirect) with only a few hundred
 queries — it's a lot, but considering a few queries per post being saved,
 not bad — and just a few short seconds on my little MacBook Air that
 could.

 Since these improvements are largely outside the menus API, I will be
 spawning individual tickets for most things.

--

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/22189#comment:5>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list