- Issue created by @alexpott
- π¬π§United Kingdom alexpott πͺπΊπ
So this works up to a point but the call to
\Drupal\Core\Path\PathValidatorInterface::getUrlIfValidWithoutAccessCheck
in\Drupal\entity_usage\EntityUsageTrackBase::processUrl
will eventually call a param convertor that will load an entity. Thinking about how to get around that... fun. - π¬π§United Kingdom alexpott πͺπΊπ
So this results in over 1000 less queries for processing 400 paragraph revisions on my large site that I'm enabling entity usage on. This approach also means we're less reliant on the entity_lru_cache module to prevent memory leaks while creating the entity usage table.
- First commit to issue fork.
- πͺπΈSpain marcoscano Barcelona, Spain
@alexpott thanks for working on this one! I left a question on the MR (to which I suspect the answer, but just in case). Apart from that, this is good to go as well IMO π
- π¬π§United Kingdom alexpott πͺπΊπ
@marcoscano thanks for the review - replied to the commment.
-
marcoscano β
committed e19fd6c9 on 8.x-2.x authored by
alexpott β
Issue #3502795: Stop loading entities when we do not need to
-
marcoscano β
committed e19fd6c9 on 8.x-2.x authored by
alexpott β
Automatically closed - issue fixed for 2 weeks with no activity.