Type: New Feature
Affects Version/s: 3.0.11
Fix Version/s: 3.0.12
When a key has an ENUM field, TableNibbler needs to wrap the ENUM field in a CONCAT stamentent because of the way MySQL handle those fields.
According to the documentation: http://dev.mysql.com/doc/refman/5.6/en/enum.html#enum-sorting
"ENUM values are sorted based on their index numbers, which depend on the order in which the enumeration members were listed in the column specification. For example, 'b' sorts before 'a' for ENUM('b', 'a'). The empty string sorts before nonempty strings, and NULL values sort before all other enumeration values.
To prevent unexpected results when using the ORDER BY clause on an ENUM column, use one of these techniques:
Specify the ENUM list in alphabetic order.
Make sure that the column is sorted lexically rather than by index number by coding ORDER BY CAST(col AS CHAR) or ORDER BY CONCAT(col)."
Since in the example the features column was defined as `feature` enum('FOO','BAR','BAT','BAZ','CAT','DOG','DERP','HERP','VANILLA','CHOCOLATE','MINT') (non-alphabetical order), pt-osc's Nibbler fails to calculate the boundaries for each chunk.
To avoid that behavior, we need these changes:
- Make TableParser to detect if the ENUM items in a table definition are already sorted or not.
- If they are not sorted:
- If --force-concat-enums was provided, continue using a CONCAT to wrap ENUM fields
- If -
force-concat-enumswas NOT provided, die with a message explaining that there are ENUM fields with unsorted items and that will cause a noticeable slowdown, and tell the user he should specify -force-concat-enums
- If they are sorted, continue as usual NOT using CONCAT to wrap the ENUM fields