[wp-trac] [WordPress Trac] #40351: Term post re-counts scale poorly, are common and difficult to avoid

[WordPress Trac]

#40351: Term post re-counts scale poorly, are common and difficult to avoid
-------------------------------------------------+-------------------------
 Reporter:  mattoperry                           |       Owner:  whyisjake
     Type:  enhancement                          |      Status:  reopened
 Priority:  normal                               |   Milestone:  5.6
Component:  Taxonomy                             |     Version:  4.8
 Severity:  normal                               |  Resolution:
 Keywords:  needs-dev-note has-patch has-unit-   |     Focuses:
  tests                                          |  performance
-------------------------------------------------+-------------------------

Comment (by peterwilsoncc):

 About to commit [https://github.com/WordPress/wordpress-develop/pull/602
 pull request 602] after discussing with @dd32 to confirm it would fix the
 warnings on meta properties.

 Something that has occurred to me is that it's not possible to use a
 custom callback for full counts (the `update_count_callback` argument) and
 the default callback for partial counts (the `update_count_by_callback`
 argument).

 A developer defining either argument with the default callback will cause
 an infinite loop. This has been the case for as long as
 `update_count_callback` has existed and I am unable to see a situation in
 which a developer would need a custom callback for full but not partial
 counts.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/40351#comment:39>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform