Commit 1a0b986e authored by Sergei Golubchik's avatar Sergei Golubchik Committed by Aleksey Midenkov

MDEV-14764 Confusing error message: Table `t1` must have at least one temporal column

compiler warning (mix of bool and enum in ?:)
parent 5dddbafa
......@@ -447,7 +447,7 @@ Warnings:
Warning 124 InnoDB rebuilding table to add column FTS_DOC_ID
create or replace table t (a int) with system versioning;
alter table t drop column a;
ERROR HY000: Table `t` must have at least one temporal column
ERROR HY000: Table `t` must have at least one versioned column
alter table t drop column a, drop column a;
ERROR 42000: Can't DROP COLUMN `a`; check that it exists
drop database test;
......
......@@ -164,7 +164,7 @@ x21 int without system versioning
create or replace table t1 (
x22 int without system versioning
) with system versioning;
ERROR HY000: Table `t1` must have at least one temporal column
ERROR HY000: Table `t1` must have at least one versioned column
create or replace table t1 (a int) with system versioning;
create table tt1 like t1;
show create table tt1;
......
......@@ -7880,7 +7880,7 @@ ER_VERS_DIFFERENT_TABLES
eng "Wrong parameters for %`s: system fields selected from different tables"
ER_VERS_TABLE_MUST_HAVE_COLUMNS
eng "Table %`s must have at least one temporal column"
eng "Table %`s must have at least one versioned column"
ER_VERS_NOT_VERSIONED
eng "Table %`s is not system-versioned"
......
......@@ -7649,7 +7649,7 @@ insert_fields(THD *thd, Name_resolution_context *context, const char *db_name,
if ((field= field_iterator.field()) &&
(field->vers_sys_field() ?
field->vers_sys_invisible(thd) :
field->invisible))
field->invisible != VISIBLE))
{
continue;
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment