[wp-trac] [WordPress Trac] #18369: Pre-cache post_meta for nav_menu_items rather than querying one at a time

[WordPress Trac]

#18369: Pre-cache post_meta for nav_menu_items rather than querying one at a time
-----------------------------+------------------
 Reporter:  mitchoyoshitaka  |       Owner:
     Type:  defect (bug)     |      Status:  new
 Priority:  normal           |   Milestone:  3.3
Component:  Performance      |     Version:
 Severity:  minor            |  Resolution:
 Keywords:  has-patch        |
-----------------------------+------------------

Comment (by mitchoyoshitaka):

 Replying to [comment:2 nacin]:
 > Are these front-end queries? Cause this doesn't look right at all. The
 198/142 look more like the admin.

 The ridiculous number of queries was a combination of the admin bar
 loading, a couple complex nav menus (which I'm now caching), and a bug
 with W3 Total Cache which I'm reporting. It's a site still early in
 development but yes, client side. Now down to 32 queries.

 > [attachment:18369.2.diff] significantly reduces the number of queries
 that are run on the admin -- rough estimate, by about two-thirds. More
 optimizations are likely possible.

 Why are you keeping update_post_term_cache false on the client side?

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