See title.

We need to shorten the index on the single fields (array('field', 50) or similar) or possibly just remove the third one from the index, that is not going to be used frequently anyway....

CommentFileSizeAuthor
#5 tmgmt-remote_identifiers_update-1902830-1.patch1 KBblueminds
#2 tmgmt-shorten_keys-1902830-1.patch997 bytesblueminds

Comments

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Priority: Normal » Major

Can you have a look at this? This prevents installation on e.g. simplytest.me for example, which afaik uses MyISAM.

blueminds’s picture

blueminds commented
Status: Active » Needs review
StatusFileSize
new tmgmt-shorten_keys-1902830-1.patch997 bytes

Based on OHT and Gengo (storing numeric vals in those fields and I do not expect this will be much different for other services) I shortened the fields to 125 and removed the third one from the index.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

Changed to 127 characters and committed.

We discussed also making just the index length limited but agreed upon that nobody will probably require something that long. Even UUID's are way below those 127 characters.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Status: Fixed » Needs work

hmm, no update hook?

blueminds’s picture

blueminds commented
Status: Needs work » Needs review
StatusFileSize
new tmgmt-remote_identifiers_update-1902830-1.patch1 KB

here it comes

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

Yeah, forgot that.

Applied, run, works, committed.

Status: Fixed » Closed (fixed)

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