search

lcache / wp-lcache

A WordPress implementation of LCache
https://wordpress.org/plugins/wp-lcache/
GNU General Public License v2.0
88 stars 5 forks source link

Switch back to `utf8` for LCache tables #149

Closed danielbachhuber closed 7 years ago

danielbachhuber commented 7 years ago

Because our value column is longtext, utf8mb4 isn't necessary for storing unicode characters.

See #148

davidstrauss commented 7 years ago

This sounds reasonable to me, though I'm not sure of the benefit if there aren't any indexes on the value column.

danielbachhuber commented 7 years ago

though I'm not sure of the benefit if there aren't any indexes on the value column.

The value column was the original motivation for utf8mb4, but the change also applied to the address column (which is included in an index).