Commit f7d03048 authored by Sergei Golubchik's avatar Sergei Golubchik

Merge branch '10.0' into 10.1

parents f797ea71 6728aae3
create table t1 (a datetime,
# get_datetime_value
b int as (a > 1), # Arg_comparator
c int as (a in (1,2,3)), # in_datetime
d int as ((a,a) in ((1,1),(2,1),(NULL,1))), # cmp_item_datetime
# other issues
e int as ((a,1) in ((1,1),(2,1),(NULL,1))) # cmp_item_row::alloc_comparators()
);
Warnings:
Warning 1292 Incorrect datetime value: '1'
Warning 1292 Incorrect datetime value: '2'
Warning 1292 Incorrect datetime value: '3'
show create table t1;
Table Create Table
t1 CREATE TABLE `t1` (
`a` datetime DEFAULT NULL,
`b` int(11) AS (a > 1) VIRTUAL,
`c` int(11) AS (a in (1,2,3)) VIRTUAL,
`d` int(11) AS ((a,a) in ((1,1),(2,1),(NULL,1))) VIRTUAL,
`e` int(11) AS ((a,1) in ((1,1),(2,1),(NULL,1))) VIRTUAL
) ENGINE=MyISAM DEFAULT CHARSET=latin1
Warnings:
Warning 1292 Incorrect datetime value: '1'
Warning 1292 Incorrect datetime value: '2'
Warning 1292 Incorrect datetime value: '3'
insert t1 (a) values ('2010-10-10 10:10:10');
select * from t1;
a b c d e
2010-10-10 10:10:10 1 0 0 NULL
Warnings:
Warning 1292 Incorrect datetime value: '1'
Warning 1292 Incorrect datetime value: '1'
Warning 1292 Incorrect datetime value: '2'
Warning 1292 Incorrect datetime value: '1'
Warning 1292 Incorrect datetime value: '1'
Warning 1292 Incorrect datetime value: '2'
select * from t1;
a b c d e
2010-10-10 10:10:10 1 0 0 NULL
Warnings:
Warning 1292 Incorrect datetime value: '1'
Warning 1292 Incorrect datetime value: '2'
Warning 1292 Incorrect datetime value: '1'
Warning 1292 Incorrect datetime value: '1'
Warning 1292 Incorrect datetime value: '2'
drop table t1;
create table t1 (a datetime,
b datetime as (least(a,1)) # Item_func_min_max::get_date
);
insert t1 (a) values ('2010-10-10 10:10:10');
select * from t1;
a b
2010-10-10 10:10:10 0000-00-00 00:00:00
Warnings:
Warning 1292 Incorrect datetime value: '1'
select * from t1;
a b
2010-10-10 10:10:10 0000-00-00 00:00:00
Warnings:
Warning 1292 Incorrect datetime value: '1'
drop table t1;
#
# This tests various issues when vcol items allocate memory (e.g. more items)
# not in the TABLE::expr_arena.
#
#
# MDEV-9690 concurrent queries with virtual columns crash in temporal code
#
create table t1 (a datetime,
# get_datetime_value
b int as (a > 1), # Arg_comparator
c int as (a in (1,2,3)), # in_datetime
d int as ((a,a) in ((1,1),(2,1),(NULL,1))), # cmp_item_datetime
# other issues
e int as ((a,1) in ((1,1),(2,1),(NULL,1))) # cmp_item_row::alloc_comparators()
);
show create table t1;
connect con1, localhost, root;
insert t1 (a) values ('2010-10-10 10:10:10');
select * from t1;
disconnect con1;
connection default;
select * from t1;
drop table t1;
connect con1, localhost, root;
create table t1 (a datetime,
b datetime as (least(a,1)) # Item_func_min_max::get_date
);
insert t1 (a) values ('2010-10-10 10:10:10');
select * from t1;
disconnect con1;
connection default;
select * from t1;
drop table t1;
...@@ -620,7 +620,7 @@ int Arg_comparator::set_compare_func(Item_func_or_sum *item, Item_result type) ...@@ -620,7 +620,7 @@ int Arg_comparator::set_compare_func(Item_func_or_sum *item, Item_result type)
int Arg_comparator::set_cmp_func(Item_func_or_sum *owner_arg, int Arg_comparator::set_cmp_func(Item_func_or_sum *owner_arg,
Item **a1, Item **a2) Item **a1, Item **a2)
{ {
thd= current_thd; THD *thd= current_thd;
owner= owner_arg; owner= owner_arg;
set_null= set_null && owner_arg; set_null= set_null && owner_arg;
a= a1; a= a1;
...@@ -752,12 +752,10 @@ get_datetime_value(THD *thd, Item ***item_arg, Item **cache_arg, ...@@ -752,12 +752,10 @@ get_datetime_value(THD *thd, Item ***item_arg, Item **cache_arg,
if (cache_arg && item->const_item() && if (cache_arg && item->const_item() &&
!(item->type() == Item::CACHE_ITEM && item->cmp_type() == TIME_RESULT)) !(item->type() == Item::CACHE_ITEM && item->cmp_type() == TIME_RESULT))
{ {
Query_arena backup; if (!thd)
Query_arena *save_arena= thd->switch_to_arena_for_cached_items(&backup); thd= current_thd;
Item_cache_temporal *cache= new (thd->mem_root) Item_cache_temporal(thd, f_type);
if (save_arena)
thd->set_query_arena(save_arena);
Item_cache_temporal *cache= new (thd->mem_root) Item_cache_temporal(thd, f_type);
cache->store_packed(value, item); cache->store_packed(value, item);
*cache_arg= cache; *cache_arg= cache;
*item_arg= cache_arg; *item_arg= cache_arg;
...@@ -792,12 +790,12 @@ int Arg_comparator::compare_temporal(enum_field_types type) ...@@ -792,12 +790,12 @@ int Arg_comparator::compare_temporal(enum_field_types type)
owner->null_value= 1; owner->null_value= 1;
/* Get DATE/DATETIME/TIME value of the 'a' item. */ /* Get DATE/DATETIME/TIME value of the 'a' item. */
a_value= get_datetime_value(thd, &a, &a_cache, type, &a_is_null); a_value= get_datetime_value(0, &a, &a_cache, type, &a_is_null);
if (a_is_null) if (a_is_null)
return -1; return -1;
/* Get DATE/DATETIME/TIME value of the 'b' item. */ /* Get DATE/DATETIME/TIME value of the 'b' item. */
b_value= get_datetime_value(thd, &b, &b_cache, type, &b_is_null); b_value= get_datetime_value(0, &b, &b_cache, type, &b_is_null);
if (b_is_null) if (b_is_null)
return -1; return -1;
...@@ -815,10 +813,10 @@ int Arg_comparator::compare_e_temporal(enum_field_types type) ...@@ -815,10 +813,10 @@ int Arg_comparator::compare_e_temporal(enum_field_types type)
longlong a_value, b_value; longlong a_value, b_value;
/* Get DATE/DATETIME/TIME value of the 'a' item. */ /* Get DATE/DATETIME/TIME value of the 'a' item. */
a_value= get_datetime_value(thd, &a, &a_cache, type, &a_is_null); a_value= get_datetime_value(0, &a, &a_cache, type, &a_is_null);
/* Get DATE/DATETIME/TIME value of the 'b' item. */ /* Get DATE/DATETIME/TIME value of the 'b' item. */
b_value= get_datetime_value(thd, &b, &b_cache, type, &b_is_null); b_value= get_datetime_value(0, &b, &b_cache, type, &b_is_null);
return a_is_null || b_is_null ? a_is_null == b_is_null return a_is_null || b_is_null ? a_is_null == b_is_null
: a_value == b_value; : a_value == b_value;
} }
...@@ -3755,7 +3753,7 @@ uchar *in_datetime::get_value(Item *item) ...@@ -3755,7 +3753,7 @@ uchar *in_datetime::get_value(Item *item)
Item **tmp_item= lval_cache ? &lval_cache : &item; Item **tmp_item= lval_cache ? &lval_cache : &item;
enum_field_types f_type= enum_field_types f_type=
tmp_item[0]->field_type_for_temporal_comparison(warn_item); tmp_item[0]->field_type_for_temporal_comparison(warn_item);
tmp.val= get_datetime_value(thd, &tmp_item, &lval_cache, f_type, &is_null); tmp.val= get_datetime_value(0, &tmp_item, &lval_cache, f_type, &is_null);
if (item->null_value) if (item->null_value)
return 0; return 0;
tmp.unsigned_flag= 1L; tmp.unsigned_flag= 1L;
...@@ -4020,7 +4018,7 @@ void cmp_item_datetime::store_value(Item *item) ...@@ -4020,7 +4018,7 @@ void cmp_item_datetime::store_value(Item *item)
Item **tmp_item= lval_cache ? &lval_cache : &item; Item **tmp_item= lval_cache ? &lval_cache : &item;
enum_field_types f_type= enum_field_types f_type=
tmp_item[0]->field_type_for_temporal_comparison(warn_item); tmp_item[0]->field_type_for_temporal_comparison(warn_item);
value= get_datetime_value(thd, &tmp_item, &lval_cache, f_type, &is_null); value= get_datetime_value(0, &tmp_item, &lval_cache, f_type, &is_null);
m_null_value= item->null_value; m_null_value= item->null_value;
} }
......
...@@ -55,7 +55,6 @@ class Arg_comparator: public Sql_alloc ...@@ -55,7 +55,6 @@ class Arg_comparator: public Sql_alloc
Arg_comparator *comparators; // used only for compare_row() Arg_comparator *comparators; // used only for compare_row()
double precision; double precision;
/* Fields used in DATE/DATETIME comparison. */ /* Fields used in DATE/DATETIME comparison. */
THD *thd;
Item *a_cache, *b_cache; // Cached values of a and b items Item *a_cache, *b_cache; // Cached values of a and b items
// when one of arguments is NULL. // when one of arguments is NULL.
int set_compare_func(Item_func_or_sum *owner, Item_result type); int set_compare_func(Item_func_or_sum *owner, Item_result type);
...@@ -70,12 +69,12 @@ class Arg_comparator: public Sql_alloc ...@@ -70,12 +69,12 @@ class Arg_comparator: public Sql_alloc
Arg_comparator(): m_compare_type(STRING_RESULT), Arg_comparator(): m_compare_type(STRING_RESULT),
m_compare_collation(&my_charset_bin), m_compare_collation(&my_charset_bin),
set_null(TRUE), comparators(0), thd(0), set_null(TRUE), comparators(0),
a_cache(0), b_cache(0) {}; a_cache(0), b_cache(0) {};
Arg_comparator(Item **a1, Item **a2): a(a1), b(a2), Arg_comparator(Item **a1, Item **a2): a(a1), b(a2),
m_compare_type(STRING_RESULT), m_compare_type(STRING_RESULT),
m_compare_collation(&my_charset_bin), m_compare_collation(&my_charset_bin),
set_null(TRUE), comparators(0), thd(0), set_null(TRUE), comparators(0),
a_cache(0), b_cache(0) {}; a_cache(0), b_cache(0) {};
public: public:
...@@ -1194,15 +1193,13 @@ class in_longlong :public in_vector ...@@ -1194,15 +1193,13 @@ class in_longlong :public in_vector
class in_datetime :public in_longlong class in_datetime :public in_longlong
{ {
public: public:
THD *thd;
/* An item used to issue warnings. */ /* An item used to issue warnings. */
Item *warn_item; Item *warn_item;
/* Cache for the left item. */ /* Cache for the left item. */
Item *lval_cache; Item *lval_cache;
in_datetime(Item *warn_item_arg, uint elements) in_datetime(Item *warn_item_arg, uint elements)
:in_longlong(elements), thd(current_thd), warn_item(warn_item_arg), :in_longlong(elements), warn_item(warn_item_arg), lval_cache(0) {};
lval_cache(0) {};
void set(uint pos,Item *item); void set(uint pos,Item *item);
uchar *get_value(Item *item); uchar *get_value(Item *item);
Item *create_item(THD *thd); Item *create_item(THD *thd);
...@@ -1372,14 +1369,13 @@ class cmp_item_datetime : public cmp_item_scalar ...@@ -1372,14 +1369,13 @@ class cmp_item_datetime : public cmp_item_scalar
{ {
longlong value; longlong value;
public: public:
THD *thd;
/* Item used for issuing warnings. */ /* Item used for issuing warnings. */
Item *warn_item; Item *warn_item;
/* Cache for the left item. */ /* Cache for the left item. */
Item *lval_cache; Item *lval_cache;
cmp_item_datetime(Item *warn_item_arg) cmp_item_datetime(Item *warn_item_arg)
:thd(current_thd), warn_item(warn_item_arg), lval_cache(0) {} : warn_item(warn_item_arg), lval_cache(0) {}
void store_value(Item *item); void store_value(Item *item);
int cmp(Item *arg); int cmp(Item *arg);
int compare(cmp_item *ci); int compare(cmp_item *ci);
...@@ -2584,4 +2580,3 @@ extern Ge_creator ge_creator; ...@@ -2584,4 +2580,3 @@ extern Ge_creator ge_creator;
extern Le_creator le_creator; extern Le_creator le_creator;
#endif /* ITEM_CMPFUNC_INCLUDED */ #endif /* ITEM_CMPFUNC_INCLUDED */
...@@ -2767,7 +2767,6 @@ void Item_func_min_max::fix_length_and_dec() ...@@ -2767,7 +2767,6 @@ void Item_func_min_max::fix_length_and_dec()
decimals=0; decimals=0;
max_length=0; max_length=0;
maybe_null=0; maybe_null=0;
thd= current_thd;
Item_result tmp_cmp_type= args[0]->cmp_type(); Item_result tmp_cmp_type= args[0]->cmp_type();
uint string_type_count= 0; uint string_type_count= 0;
uint temporal_type_count= 0; uint temporal_type_count= 0;
...@@ -2909,10 +2908,8 @@ bool Item_func_min_max::get_date(MYSQL_TIME *ltime, ulonglong fuzzy_date) ...@@ -2909,10 +2908,8 @@ bool Item_func_min_max::get_date(MYSQL_TIME *ltime, ulonglong fuzzy_date)
longlong res= args[i]->val_temporal_packed(Item_func_min_max::field_type()); longlong res= args[i]->val_temporal_packed(Item_func_min_max::field_type());
/* Check if we need to stop (because of error or KILL) and stop the loop */ /* Check if we need to stop (because of error or KILL) and stop the loop */
if (thd->is_error() || args[i]->null_value) if (args[i]->null_value)
{
return (null_value= 1); return (null_value= 1);
}
if (i == 0 || (res < min_max ? cmp_sign : -cmp_sign) > 0) if (i == 0 || (res < min_max ? cmp_sign : -cmp_sign) > 0)
min_max= res; min_max= res;
......
...@@ -1068,7 +1068,6 @@ class Item_func_min_max :public Item_hybrid_func ...@@ -1068,7 +1068,6 @@ class Item_func_min_max :public Item_hybrid_func
{ {
String tmp_value; String tmp_value;
int cmp_sign; int cmp_sign;
THD *thd;
public: public:
Item_func_min_max(THD *thd, List<Item> &list, int cmp_sign_arg): Item_func_min_max(THD *thd, List<Item> &list, int cmp_sign_arg):
Item_hybrid_func(thd, list), cmp_sign(cmp_sign_arg) Item_hybrid_func(thd, list), cmp_sign(cmp_sign_arg)
......
...@@ -1085,7 +1085,6 @@ THD::THD(bool is_wsrep_applier) ...@@ -1085,7 +1085,6 @@ THD::THD(bool is_wsrep_applier)
m_internal_handler= NULL; m_internal_handler= NULL;
m_binlog_invoker= INVOKER_NONE; m_binlog_invoker= INVOKER_NONE;
arena_for_cached_items= 0;
memset(&invoker_user, 0, sizeof(invoker_user)); memset(&invoker_user, 0, sizeof(invoker_user));
memset(&invoker_host, 0, sizeof(invoker_host)); memset(&invoker_host, 0, sizeof(invoker_host));
prepare_derived_at_open= FALSE; prepare_derived_at_open= FALSE;
......
...@@ -3837,27 +3837,7 @@ class THD :public Statement, ...@@ -3837,27 +3837,7 @@ class THD :public Statement,
} }
} }
private:
/*
This reference points to the table arena when the expression
for a virtual column is being evaluated
*/
Query_arena *arena_for_cached_items;
public: public:
void reset_arena_for_cached_items(Query_arena *new_arena)
{
arena_for_cached_items= new_arena;
}
Query_arena *switch_to_arena_for_cached_items(Query_arena *backup)
{
if (!arena_for_cached_items)
return 0;
set_n_backup_active_arena(arena_for_cached_items, backup);
return backup;
}
void clear_wakeup_ready() { wakeup_ready= false; } void clear_wakeup_ready() { wakeup_ready= false; }
/* /*
Sleep waiting for others to wake us up with signal_wakeup_ready(). Sleep waiting for others to wake us up with signal_wakeup_ready().
......
...@@ -6890,7 +6890,9 @@ int update_virtual_fields(THD *thd, TABLE *table, ...@@ -6890,7 +6890,9 @@ int update_virtual_fields(THD *thd, TABLE *table,
int error __attribute__ ((unused))= 0; int error __attribute__ ((unused))= 0;
DBUG_ASSERT(table && table->vfield); DBUG_ASSERT(table && table->vfield);
thd->reset_arena_for_cached_items(table->expr_arena); Query_arena backup_arena;
thd->set_n_backup_active_arena(table->expr_arena, &backup_arena);
/* Iterate over virtual fields in the table */ /* Iterate over virtual fields in the table */
for (vfield_ptr= table->vfield; *vfield_ptr; vfield_ptr++) for (vfield_ptr= table->vfield; *vfield_ptr; vfield_ptr++)
{ {
...@@ -6908,7 +6910,7 @@ int update_virtual_fields(THD *thd, TABLE *table, ...@@ -6908,7 +6910,7 @@ int update_virtual_fields(THD *thd, TABLE *table,
DBUG_PRINT("info", ("field '%s' - skipped", vfield->field_name)); DBUG_PRINT("info", ("field '%s' - skipped", vfield->field_name));
} }
} }
thd->reset_arena_for_cached_items(0); thd->restore_active_arena(table->expr_arena, &backup_arena);
DBUG_RETURN(0); DBUG_RETURN(0);
} }
......
...@@ -97,7 +97,7 @@ bool ZIPUTIL::WildMatch(PSZ pat, PSZ str) { ...@@ -97,7 +97,7 @@ bool ZIPUTIL::WildMatch(PSZ pat, PSZ str) {
if (!*++pat) return TRUE; if (!*++pat) return TRUE;
goto loopStart; goto loopStart;
default: default:
if (mapCaseTable[*s] != mapCaseTable[*p]) if (mapCaseTable[(unsigned)*s] != mapCaseTable[(unsigned)*p])
goto starCheck; goto starCheck;
break; break;
} /* endswitch */ } /* endswitch */
...@@ -151,7 +151,7 @@ int ZIPUTIL::findEntry(PGLOBAL g, bool next) ...@@ -151,7 +151,7 @@ int ZIPUTIL::findEntry(PGLOBAL g, bool next)
if (rc == UNZ_END_OF_LIST_OF_FILE) if (rc == UNZ_END_OF_LIST_OF_FILE)
return RC_EF; return RC_EF;
else if (rc != UNZ_OK) { else if (rc != UNZ_OK) {
sprintf(g->Message, "unzGoToNextFile rc = ", rc); sprintf(g->Message, "unzGoToNextFile rc = %d", rc);
return RC_FX; return RC_FX;
} // endif rc } // endif rc
...@@ -261,7 +261,7 @@ bool ZIPUTIL::OpenTable(PGLOBAL g, MODE mode, char *fn) ...@@ -261,7 +261,7 @@ bool ZIPUTIL::OpenTable(PGLOBAL g, MODE mode, char *fn)
fp->Memory = memory; fp->Memory = memory;
fp->Mode = mode; fp->Mode = mode;
fp->File = this; fp->File = this;
fp->Handle = NULL; fp->Handle = 0;
} // endif fp } // endif fp
} else } else
...@@ -297,7 +297,7 @@ bool ZIPUTIL::openEntry(PGLOBAL g) ...@@ -297,7 +297,7 @@ bool ZIPUTIL::openEntry(PGLOBAL g)
memory = new char[size + 1]; memory = new char[size + 1];
if ((rc = unzReadCurrentFile(zipfile, memory, size)) < 0) { if ((rc = unzReadCurrentFile(zipfile, memory, size)) < 0) {
sprintf(g->Message, "unzReadCurrentFile rc = ", rc); sprintf(g->Message, "unzReadCurrentFile rc = %d", rc);
unzCloseCurrentFile(zipfile); unzCloseCurrentFile(zipfile);
free(memory); free(memory);
memory = NULL; memory = NULL;
......
...@@ -129,8 +129,9 @@ extern "C" { ...@@ -129,8 +129,9 @@ extern "C" {
#endif #endif
#endif #endif
#ifndef OF
#define OF(args) args
#endif
typedef voidpf (ZCALLBACK *open_file_func) OF((voidpf opaque, const char* filename, int mode)); typedef voidpf (ZCALLBACK *open_file_func) OF((voidpf opaque, const char* filename, int mode));
typedef uLong (ZCALLBACK *read_file_func) OF((voidpf opaque, voidpf stream, void* buf, uLong size)); typedef uLong (ZCALLBACK *read_file_func) OF((voidpf opaque, voidpf stream, void* buf, uLong size));
......
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