The primary index remains as sid rather than sid, ssid after an upgrade from d6 to d7.

CommentFileSizeAuthor
#2 missing_index_on_upgrade.patch837 bytesmarcingy
missing_index_on_upgrade.patch724 bytesmarcingy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented

Good catch, but please fold that into update_fix_d7_requirements(). I prefer we keep all the related updates at the same place.

marcingy’s picture

marcingy CreditAttribution: marcingy commented
FileSize
missing_index_on_upgrade.patch837 bytes

Moving change to update_fix_d7_requirements()

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

Looks good.

All these alter table statements are gonna be brutully slow on a large sessions table on InnoDB. Sigh.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

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