- Issue created by @alexpott
- Status changed to Needs review
11 months ago 12:54pm 19 January 2024 - π¬π§United Kingdom alexpott πͺπΊπ
If you install standard from config this saves 0.05 MB - so not massive but not nothing. Compare
[notice] Performed install task: install_config_import_batch [7.25 sec, 72.28 MB]
with
[notice] Performed install task: install_config_import_batch [7.05 sec, 72.23 MB]
On a real site that has a pile of config and modules installed and has been on Drupal for quite a while... compare
[notice] Performed install task: install_config_import_batch [57.93 sec, 908.19 MB]
with
[notice] Performed install task: install_config_import_batch [58.34 sec, 896.32 MB]
So that's nearly 12 MB!!! Note that the times vary quite a lot on repeated runs but the memory usage is exactly the same. - Status changed to RTBC
11 months ago 1:44pm 19 January 2024 - π§πͺBelgium borisson_ Mechelen, π§πͺ
This looks great, especially when looking at lines of code per mb saved.
- π¬π§United Kingdom longwave UK
This doesn't seem worth the effort of backporting so left this one in 11.x only.
Committed and pushed 6cbfa8c2de to 11.x. Thanks!
-
longwave β
committed 6cbfa8c2 on 11.x
Issue #3415821 by alexpott: StorageComparer should not wrap...
-
longwave β
committed 6cbfa8c2 on 11.x
- Status changed to Fixed
11 months ago 10:53pm 24 January 2024 Automatically closed - issue fixed for 2 weeks with no activity.