Use batch in field_sql_storage_update_8000() [#1943494]

Testing the upgrade path on a large site with 600k nodes and close to 300 fields, this function takes longer than the rest of the update functions combined. Some single alter table queries took over 100s and this is not a slow system (It did run the core tests in 14m 30s this week :p).

We can't do much about the execution time of the single queries but we should probably split it up and do a single table per call. It was actually able to run through with a max execution time of 30s as only a tiny fraction of the total time was spent in PHP but some servers might limit the total time. You're probably not running such a site on a server that does but still.

Comment	File	Size	Author
#2	1943494-2.patch	3.03 KB	swentel
#2

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

chx CreditAttribution: chx commented 19 March 2013 at 02:45

While at it:

      // Do not attempt to rename the 'language' column for fields that already
      // contain it (created during the upgrade before this update function).
      if (db_field_exists($table, 'language')) {

I strongly suspect this wanted to be langcode column: // Do not attempt to rename the 'langcode' column for fields that already otherwise the sentence + code is contradicting.