Since tlid is auto_increment column, it alone suffices as a primary key. Nid or (nid, tlid) should be indexes if needed. I did not actually realized this until I tried to convert the database engine to InnoDB, which requires that an auto_increment column be the primary key of the table.

Comments

mradcliffe’s picture

Version: 6.x-1.1 » 6.x-1.x-dev

Oh, crap. Thank you for reporting this.

I'll need to make an upgrade path as well.

mradcliffe’s picture

Status: Active » Needs review
StatusFileSize
new997 bytes

Here's a rough patch. I don't have any tests on this module since it's a bit old.

mradcliffe’s picture

Priority: Normal » Major
Status: Needs review » Fixed

That patch didn't work in the upgrade path, but I fixed it following http://drupal.org/node/190027#comment-1148400.

Making this a speedy push as well.

Status: Fixed » Closed (fixed)

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