I'm thinking it might be better to use different groups for each query type, ie network-queries, post-queries, etc. This will allow for more precise targeting of affected groups in cache plugins, either for the purpose of making the groups non-persistent or doing group deletes.

LGTM, thanks!

@tillkruss Technically that looks good and IMO reasonable, though I could see how for certain environments this could be a quite impactful change (for better or for worse).

I'm not 100% convinced, but I feel like this would be a candidate that requires needs-dev-note on the Trac ticket. cc @peterwilsoncc @spacedmonkey

Looks good to me 👍

Note: It appears that we use both underscores and dashes in cache group names in core. At a glance, underscores seem more common, so I went back and forth for a bit on whether to suggest using underscores here, e.g. post_queries. But we're not super consistent with this, so I guess either way is fine 🙂

I really like this change. I think it opens up lots of doors for changing caching behaviours. It will also allows for better measuring for how much of caches query based caches.

I think it change is too big for late WP 6.2, but should be committed early 6.3.