- Issue created by @larowlan
- Merge request !1163Issue #3530907: Improve cold cache performance of /components end point β (Merged) created by larowlan
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
This was causing cypress fails for me locally due to the time out.
MR improves performance by 80%
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Yep, I knew this: #3515510-2: Improve backend API performance: add caching to shape matching β π that's why I moved it to the issue queue component :) This essentially does exactly what I intended π
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
There's plenty more low-hanging fruit, see https://git.drupalcode.org/project/experience_builder/-/merge_requests/1....
There's also π [PP-1] Refactor GeneratedFieldExplicitInputUxComponentSourceBase and FieldForComponentSuggester to need only SDC's ComponentMetadata, not SDC plugin instances Postponed to decouple this from SDCs.
But here's the real kicker: we don't need any of this until a
ContentTemplate
UI exists! So I've been wanting to removedynamic_prop_sources
from the/xb/api/v0/config/component
response for a good while. Which is why π Move `PropSourceEndpointTest` into new `XbConfigEntityHttpApiTest::testComponent()` Active has been assigned to me for months. That will simply remove these computations from the critical path!Because the
ContentTemplate
UI should be able to just request the candidate dynamic prop sources on a per-content entity-t bundle basis. There's never a need to list them all across all entity types/bundles β I did that only for early PoC/testing purposes! - π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
-
wim leers β
committed 6890e9a6 on 0.x authored by
larowlan β
Issue #3530907 by larowlan, wim leers: Optimize DynamicPropSource...
-
wim leers β
committed 6890e9a6 on 0.x authored by
larowlan β
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ