Closed (outdated)
Project:
Drupal core
Version:
9.4.x-dev
Component:
database update system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
20 Jun 2009 at 19:47 UTC
Updated:
17 Jul 2022 at 16:06 UTC
Jump to comment: Most recent
Comments
Comment #1
dries commentedBefore something like this gets committed I want to study why we load all the install files (other than during installation).
Comment #2
alexanderpas commented- to verify weither we need to run database updates. drupal_get_schema_versions() trough system_requirements() which is an implementation of hook_requirements().
- during installation via install.php
- during updates via update.php
Comment #3
pasqualleOk, so this did not happen for D7.
Does it makes sense for D8? I guess it depends on changes with Migrate in core
Comment #4
catchThere's a real memory/CPU cost to having the hook_update_N() functions in .install files when calling hook_requirements(). Because even if APC is enabled. that cache is often not warm, and then when it's warmed it's likely to be not hit again for hours or days. The number of updates is going to get less if we only put minor updates there, and not at all if we figure out migrate for minor updates. So probably not worth the change at this point, I guess 'needs more info'?
Comment #5
pasqualle#788496: Break update functions into update.inc marked as duplicate
Comment #17
smustgrave commentedClosing as outdated since #4 comment went unanswered for 9 years. If still an issue please reopen.