Multiple calls to EntryQuery::loadAuthorIds()

[leevigraham]

@brandonkelly Looking at the DB queries in the debug toolbar and I'm seeing a lot of duplicated queries which are triggered by EntryQuery::loadAuthorIds().

In my case I believe this is becuase:

1. The router is loading the entry
2. SEOmatic is loading the entry (@khalwat)
3. I'm displaying nested content blocks
4. Hyper fields linking to entries (@engram-design)
5. Refs in a ckeditor field linking to an entry
6. and a bunch of other relationships

Can these queries be cached? I know they're super fast.

Maybe I'm looking in the wrong place for optimisations and should be thinking about reducing relationships.

[brandonkelly]

Thanks for pointing that out. The idea was to reduce SQL queries by fetching all queried entries’ author IDs up front in a single query per entry query. But that doesn’t make a ton of sense when the author info isn’t actually going to be needed most of the time.

Resolved this for Craft 5.8 by lazy eager-loading entry authors from Entry::getAuthors(), for all entries that were queried with the current entry (if more than one) instead. (895662a)

[leevigraham]

Thanks @brandonkelly might have been some db caching as well but that seems to have made a significant difference in overall DB query performance:

Before:

After: