Even if I only have ~4000 nodes with nodeword data to migrate, it still creates batche operations consisting of all 20,000 of my nodes where it does nothing for most of them.

Rather than selecting every nid in the node table, we should just select nids for nodes that have data in nodewords.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

iamEAP’s picture

Mistakes in my understanding of the DB API.

iamEAP’s picture

Status: Needs review » Needs work

Nevermind, this actually doesn't appear to be solving anything.

iamEAP’s picture

Status: Needs work » Needs review
FileSize
443 bytes

Ah, the real problem was in the node_save(). Very expensive; it's better to just save the fields attached to it.

The above work may still be relevant, but this patch is a significant improvement.

Sorry for all the noise as I work through this.

valthebald’s picture

Status: Needs review » Fixed

I think this approach completely makes sense.
Committed to 7.x-2.x-dev, thank you!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.