Skip to content

Commit

Permalink
Remove secondary key from scannerpath due to key length issues.
Browse files Browse the repository at this point in the history
This will require users to revert their database, or more easily `drop
table scannerfile` if they attempted and failed updating the schema
previously.

Note, this may cause future problems, as at least my system did not
reject the key, and there may be inconsistencies between users running
master branch.
  • Loading branch information
wagnerrp committed Jul 8, 2012
1 parent 8cdf5da commit dea4880
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion mythtv/libs/libmythtv/dbcheck.cpp
Expand Up @@ -2260,7 +2260,6 @@ NULL
" `storagegroup` VARCHAR(32) NOT NULL DEFAULT 'Default',"
" `filename` VARCHAR(255) NOT NULL DEFAULT '',"
" PRIMARY KEY (`fileid`),"
" UNIQUE KEY path (`storagegroup`, `hostname`, `filename`)"
") ENGINE=MyISAM DEFAULT CHARSET=utf8;",
"CREATE TABLE videopart ("
" `fileid` BIGINT(20) UNSIGNED NOT NULL,"
Expand All @@ -2271,6 +2270,9 @@ NULL
NULL
};

// removed "UNIQUE KEY path (`storagegroup`, `hostname`, `filename`)" from
// scannerpath as a quick fix for key length constraints

if (!performActualUpdate(&updates[0], "1307", dbver))
return false;
}
Expand Down

0 comments on commit dea4880

Please sign in to comment.