Details
-
Task
-
Status: Closed (View Workflow)
-
Major
-
Resolution: Fixed
-
None
Description
During a review for MDEV-13995 Sanja suggested to have a separate class as an output parameter for val_raw_xxx(), e.g.:
bool val_raw_native(THD *thd, Raw *to); |
instead of:
bool val_raw_native(THD *thd, String *to); |
Passing a String looks redundant, as val_raw_xxx() does not need a character set.
Under terms of this task we'll:
- Extract the character-set-unaware part of String into a new class Binary_string
- Derive String from Binary_string and Charset
Then, while cleaning up the patch for MDEV-13995, we'll introduce a new class Raw and also derive it from Binary_string.
Attachments
Issue Links
- blocks
-
MDEV-13995 MAX(timestamp) returns a wrong result near DST change
-
- Closed
-
- is blocked by
-
MDEV-17907 Class Static_binary_string
-
- Closed
-
- relates to
-
MDEV-7063 Split String into logical components
-
- Stalled
-
Activity
Field | Original Value | New Value |
---|---|---|
Link |
This issue blocks |
Link |
This issue is blocked by |
issue.field.resolutiondate | 2018-12-05 16:09:03.0 | 2018-12-05 16:09:03.475 |
Fix Version/s | 10.4.1 [ 23228 ] | |
Fix Version/s | 10.4 [ 22408 ] | |
Resolution | Fixed [ 1 ] | |
Status | Open [ 1 ] | Closed [ 6 ] |
Workflow | MariaDB v3 [ 91051 ] | MariaDB v4 [ 133781 ] |