[wp-trac] [WordPress Trac] #10852: improve get_page_children

[WordPress Trac]

#10852: improve get_page_children
--------------------------+-------------------------------------------------
 Reporter:  hailin        |       Owner:  westi    
     Type:  enhancement   |      Status:  reviewing
 Priority:  normal        |   Milestone:  2.9      
Component:  Optimization  |     Version:  2.9      
 Severity:  normal        |    Keywords:  has-patch
--------------------------+-------------------------------------------------

Comment(by mihai):

 Replying to [comment:7 hailin]:
 > mihai:
 >     I'm not sure calling update_page_cache in those page functions is a
 bug - it part of the cache updating mechanism.

 With my test updating the cache took about 33% of the time. If the page
 loaded in 3 minutes with no improvement, it loaded in 1 minute with the
 improved get_page_children and in just a few seconds if I removed the
 update_page_cache call.
 It might not be a bug it it adds significant time to the loading. Still I
 can't see why I would want to update the page cache when I view the page
 list? Shouldn't this cache be updated only when a page is changed ?

 >
 > However, both Ryan and I have identified this as the biggest time
 consumption issue
 > when there are thousands of pages.
 How many pages did you have ? how much time did it take to load? Did you
 actually test by loading the page admin ?

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