system

batch

    column token - difference on: default
    declared: array('description' => 'TODO: please describe this field!', 'type' => 'varchar', 'length' => 64, 'not null' => TRUE)
    actual: array('description' => 'TODO: please describe this field!', 'type' => 'varchar', 'length' => '64', 'not null' => TRUE, 'default' => '')
    column timestamp - difference on: default
    declared: array('description' => 'TODO: please describe this field!', 'type' => 'int', 'not null' => TRUE)
    actual: array('description' => 'TODO: please describe this field!', 'type' => 'int', 'not null' => TRUE, 'default' => 0)

date_format_type

    indexes title: missing in database

registry

    column filename - difference on: default
    declared: array('description' => 'TODO: please describe this field!', 'type' => 'varchar', 'length' => 255, 'not null' => TRUE)
    actual: array('description' => 'TODO: please describe this field!', 'type' => 'varchar', 'length' => '255', 'not null' => TRUE, 'default' => '')

sessions

    column uid - difference on: default
    declared: array('description' => 'TODO: please describe this field!', 'type' => 'int', 'unsigned' => TRUE, 'not null' => TRUE)
    actual: array('description' => 'TODO: please describe this field!', 'type' => 'int', 'unsigned' => TRUE, 'not null' => TRUE, 'default' => 0)
CommentFileSizeAuthor
#10 1154820-system-schema-update-8.patch807 bytesmarcingy
#8 1154820-system-schema-update-8.diff807 bytesmarcingy
#1 system-upgrade-schema-1154820.patch656 byteslyricnz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lyricnz’s picture

lyricnz CreditAttribution: lyricnz commented
Version: 7.0 » 7.x-dev
Status: Active » Needs review
FileSize
system-upgrade-schema-1154820.patch656 bytes

Fix {date_format_type}.title and {registry}.filename.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Priority: Normal » Major
Issue tags: +D7 upgrade path
marcingy’s picture

marcingy CreditAttribution: marcingy commented
Status: Needs review » Needs work

This fails when updating on a db where the index already exist, we need to first drop the index and then recreate. May the index creation should move into update.inc.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented

Dropping and re-adding the index seems fine.

What do you mean by moving the index creation into update.inc?

marcingy’s picture

marcingy CreditAttribution: marcingy commented

Just avoid the drop and add for existing d7 sites - so run in update_prepare_d7_bootstrap.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented

Oh I see, I think I'd leave the drop and add in the new update - mainly because this is table won't have many records at all.

marcingy’s picture

marcingy CreditAttribution: marcingy commented
Status: Needs work » Needs review
FileSize
1154820-system-schema-update-8.diff807 bytes

Thanks catch. New version including the index drop.

marcingy’s picture

marcingy CreditAttribution: marcingy commented

Thanks catch. New version including the index drop.

marcingy’s picture

marcingy CreditAttribution: marcingy commented
FileSize
1154820-system-schema-update-8.patch807 bytes

File failed to upload

lyricnz’s picture

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

RTBC per #1154806-5: Comment Schema mismatch after upgrade D6 to D7

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x. Thanks!

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