Details
-
Bug
-
Status: Closed (View Workflow)
-
Major
-
Resolution: Fixed
-
None
-
None
Description
We have a function:
static inline enum enum_mysql_timestamp_type |
mysql_type_to_time_type(enum enum_field_types mysql_type) |
{
|
switch(mysql_type) { |
case MYSQL_TYPE_TIME2: |
case MYSQL_TYPE_TIME: return MYSQL_TIMESTAMP_TIME; |
case MYSQL_TYPE_TIMESTAMP2: |
case MYSQL_TYPE_TIMESTAMP: |
case MYSQL_TYPE_DATETIME2: |
case MYSQL_TYPE_DATETIME: return MYSQL_TIMESTAMP_DATETIME; |
case MYSQL_TYPE_NEWDATE: |
case MYSQL_TYPE_DATE: return MYSQL_TIMESTAMP_DATE; |
default: return MYSQL_TIMESTAMP_ERROR; |
}
|
}
|
This is not friendly to pluggable data types.
We'll remove this function and will use
Type_handler::mysql_timestamp_type() instead.