Transaction_reader not quoting enum values properly
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Drizzle |
Fix Released
|
High
|
David Shrewsbury | ||
7.0 |
Fix Released
|
High
|
David Shrewsbury |
Bug Description
The file drizzled/
As we can see below, the values for all enum columns are not quoted:
col_enum (8th item in the INSERTs), col_enum_key (10th item), etc
The log looks good, but transaction_reader is misinterpreting things.
Also see Bug#641685 - Transaction reader not properly converting all NULL values into usable SQL
INSERT INTO `c` VALUES (1,NULL,
INSERT INTO `test`.`c` (`pk`,`
statement {
type: INSERT
start_timestamp: 1284763044959043
end_timestamp: 1284763045008367
insert_header {
table_metadata {
schema_name: "test"
table_name: "c"
}
field_metadata {
type: INTEGER
name: "pk"
}
field_metadata {
type: INTEGER
name: "col_int_key"
}
field_metadata {
type: BLOB
name: "col_text_
}
field_metadata {
type: BIGINT
name: "col_bigint_key"
}
field_metadata {
type: VARCHAR
name: "col_char_
}
field_metadata {
type: BLOB
name: "col_text_key"
}
field_metadata {
type: VARCHAR
name: "col_char_
}
field_metadata {
type: ENUM
name: "col_enum"
}
field_metadata {
type: BLOB
name: "col_text_not_null"
}
field_metadata {
type: ENUM
name: "col_enum_key"
}
field_metadata {
type: BIGINT
name: "col_bigint_
}
field_metadata {
type: ENUM
name: "col_enum_not_null"
}
field_metadata {
type: INTEGER
name: "col_int_
}
field_metadata {
type: VARCHAR
name: "col_char_1024"
}
field_metadata {
type: INTEGER
name: "col_int"
}
field_metadata {
type: BLOB
name: "col_text"
}
field_metadata {
type: ENUM
name: "col_enum_
}
field_metadata {
type: VARCHAR
name: "col_char_10"
}
field_metadata {
type: BIGINT
name: "col_bigint_
}
field_metadata {
type: INTEGER
name: "col_int_not_null"
}
field_metadata {
type: VARCHAR
name: "col_char_
}
field_metadata {
type: VARCHAR
name: "col_char_
}
field_metadata {
type: VARCHAR
name: "col_char_10_key"
}
field_metadata {
type: VARCHAR
name: "col_char_1024_key"
}
field_metadata {
type: BIGINT
name: "col_bigint"
}
}
insert_data {
segment_id: 1
end_segment: true
record {
insert_value: "1"
insert_value: ""
insert_value: ""
insert_value: "2"
insert_value: "t"
insert_value: ""
insert_value: "d"
insert_value: "v"
insert_value: ""
insert_value: "h"
insert_value: "32518804059382
insert_value: "v"
insert_value: "0"
insert_value: "q"
insert_value: ""
insert_value: ""
insert_value: "n"
insert_value: "h"
insert_value: "1"
insert_value: "6"
insert_value: "look"
insert_value: "TYZGK"
insert_value: "b"
insert_value: "JRKMQ"
insert_value: ""
is_null: false
is_null: true
is_null: false
is_null: false
is_null: false
is_null: true
is_null: false
is_null: false
is_null: false
is_null: false
is_null: false
is_null: false
is_null: false
is_null: false
is_null: true
is_null: true
is_null: false
is_null: false
is_null: false
is_null: false
is_null: false
is_null: false
is_null: false
is_null: false
is_null: true
}
}
}
Related branches
- Monty Taylor: Approve
-
Diff: 71 lines (+9/-10)4 files modifieddrizzled/message/statement_transform.cc (+0/-1)
plugin/transaction_log/tests/r/null_values.result (+2/-2)
plugin/transaction_log/tests/r/replace.result (+6/-6)
plugin/transaction_log/tests/t/null_values.inc (+1/-1)
Changed in drizzle: | |
status: | New → Confirmed |
importance: | Undecided → High |
assignee: | nobody → David Shrewsbury (dshrews) |