- developers - lists.mariadb.org

Re: [Maria-developers] 1ed4683: MDEV-8260 : Issues related to concurrent CTAS
by Sergei Golubchik 09 Jun '15

09 Jun '15

Hi, Nirbhay! On Jun 02, Nirbhay Choubey wrote: > revision-id: 1ed46831ed3a1c842b45f7819edd48a233080ce4 > parent(s): 6d5b723bdc3e04978619b9673fca266e0426916f > committer: Nirbhay Choubey > branch nick: 10.0-galera.ctas > timestamp: 2015-06-02 17:59:32 -0400 > message: > > MDEV-8260 : Issues related to concurrent CTAS > > * Wait for aborted thd (victim) to release MDL locks > * Skip aborting an already aborted thd > * Defer setting OK status in case of CTAS > * Minor cosmetic changes > * Added a test case So, I'm only reviewing non-wsrep part, ok? > @@ -3735,24 +3757,16 @@ bool select_insert::send_eof() > table->file->print_error(error,MYF(0)); > DBUG_RETURN(1); > } > - char buff[160]; > - if (info.ignore) > - sprintf(buff, ER(ER_INSERT_INFO), (ulong) info.records, > - (ulong) (info.records - info.copied), > - (long) thd->get_stmt_da()->current_statement_warn_count()); > - else > - sprintf(buff, ER(ER_INSERT_INFO), (ulong) info.records, > - (ulong) (info.deleted+info.updated), > - (long) thd->get_stmt_da()->current_statement_warn_count()); > - row_count= info.copied + info.deleted + > - ((thd->client_capabilities & CLIENT_FOUND_ROWS) ? > - info.touched : info.updated); > - id= (thd->first_successful_insert_id_in_cur_stmt > 0) ? > - thd->first_successful_insert_id_in_cur_stmt : > - (thd->arg_of_last_insert_id_function ? > - thd->first_successful_insert_id_in_prev_stmt : > - (info.copied ? autoinc_value_of_last_inserted_row : 0)); > - ::my_ok(thd, row_count, id, buff); > + > + /* > + Avoid setting OK status for CREATE .. SELECT as it may fail during commit. > + */ > + if (!((thd->lex->sql_command == SQLCOM_CREATE_TABLE) && > + ((&thd->lex->select_lex)->item_list.elements))) { > + prepare_ok_info(thd); > + ::my_ok(thd, ok.row_count, ok.id, ok.message); > + } > + > DBUG_RETURN(0); > } I think we've discussed a slightly different fix, where the parent class (select_insert) wouldn't need to check thd->lex->sql_command. Like, create two methods select_insert::prepare_eof select_insert::send_ok_packet the first method will have everything of the current select_insert::send_eof but without ::my_ok part. Then select_insert::send_eof() will be just bool select_insert::send_eof() { return prepare_eof() || send_ok_packet(); } and select_create::send_eof() will do prepare_eof(), commit and other stuff, send_ok_packet(). In this case you won't need to test for thd->lex->sql_command in the parent class. And you won't need prepare_ok_info(), it'll be inside send_ok_packet(). Regards, Sergei

2 1

Re: [Maria-developers] 87088b9: MDEV-8211 plugins.server_audit fails sporadically in buildbot.
by Sergei Golubchik 08 Jun '15

08 Jun '15

Hi, Holyfoot! On Jun 08, holyfoot(a)askmonty.org wrote: > revision-id: 87088b91f7dfa49a1946454d347d823dab643569 > parent(s): 96b37035e4045a27f64810c2d946bad2d1453a80 > committer: Alexey Botchkov > branch nick: server > timestamp: 2015-06-08 21:44:13 +0500 > message: > > MDEV-8211 plugins.server_audit fails sporadically in buildbot. > This test also should be fixed - delay added so the connection > event doesn't happen before the query. > > --- > mysql-test/suite/plugins/t/thread_pool_server_audit.test | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/mysql-test/suite/plugins/t/thread_pool_server_audit.test b/mysql-test/suite/plugins/t/thread_pool_server_audit.test > index 1a2d729..2c749e2 100644 > --- a/mysql-test/suite/plugins/t/thread_pool_server_audit.test > +++ b/mysql-test/suite/plugins/t/thread_pool_server_audit.test > @@ -13,6 +13,7 @@ set global server_audit_incl_users=null; > set global server_audit_file_path='server_audit.log'; > set global server_audit_output_type=file; > set global server_audit_logging=on; > +--sleep 2 > connect (con1,localhost,root,,mysql); > connection default; > disconnect con1; This is not a good idea. First, delays make mysql-test slower. And most often - unnecessary slower, as the test succeeds without a delay too. But also delays are unreliable, on a slow box two seconds may be not enough. Don't use delays, please, in this case you can wait for the query to complete before opening a new connection. Regards, Sergei

1 0

Re: [Maria-developers] [Commits] 5ccbfa0: MDEV-7445: Server crash with Signal 6
by Sergey Petrunia 08 Jun '15

08 Jun '15

On Thu, Apr 23, 2015 at 07:11:09PM +0200, sanja(a)mariadb.com wrote: > revision-id: 5ccbfa02d4da91c3f9080dc1b1627d19cf40eccd > parent(s): c7efc0315b51f214b6423ae4dcb4ddeb1b14e91a > committer: Oleksandr Byelkin > branch nick: server > timestamp: 2015-04-23 19:11:06 +0200 > message: > > MDEV-7445: Server crash with Signal 6 > > Problem was in rewriting left expression which had 2 references on it. Solved with making subselect reference main. > > Item_in_optimized can have not Item_in_subselect reference in left part so type casting with no check is dangerous. > > Item::cols() should be checked after Item::fix_fields(). > > --- ... > diff --git a/sql/item_cmpfunc.cc b/sql/item_cmpfunc.cc > index 998cb1c..51a38f0 100644 > --- a/sql/item_cmpfunc.cc > +++ b/sql/item_cmpfunc.cc > @@ -1442,9 +1442,25 @@ bool Item_in_optimizer::eval_not_null_tables(uchar *opt_arg) > bool Item_in_optimizer::fix_left(THD *thd, Item **ref) > { > DBUG_ENTER("Item_in_optimizer::fix_left"); > - if ((!args[0]->fixed && args[0]->fix_fields(thd, args)) || > - (!cache && !(cache= Item_cache::get_cache(args[0])))) > + Item **ref0= args; > + if (args[1]->type() == Item::SUBSELECT_ITEM && > + ((Item_subselect *)args[1])->is_in_predicate()) > + { > + /* > + left_expr->fix_fields() may cause left_expr to be substituted for > + another item. (e.g. an Item_field may be changed into Item_ref). This > + transformation is undone at the end of statement execution (e.g. the > + Item_ref is deleted). However, Item_in_optimizer::args[0] may keep > + the pointer to the post-transformation item. Because of that, on the > + next execution we need to copy args[1]->left_expr again. > + */ > + ref0= &(((Item_in_subselect *)args[1])->left_expr); > + args[0]= ref0[0]; > + } > + if ((!args[0]->fixed && args[0]->fix_fields(thd, ref0)) || > + (!cache && !(cache= Item_cache::get_cache(ref0[0])))) > DBUG_RETURN(1); > + args[0]= ref0[0]; > DBUG_PRINT("info", ("actual fix fields")); > > cache->setup(args[0]); > @@ -1500,6 +1516,16 @@ bool Item_in_optimizer::fix_left(THD *thd, Item **ref) > bool Item_in_optimizer::fix_fields(THD *thd, Item **ref) > { > DBUG_ASSERT(fixed == 0); > + Item_subselect *sub= 0; > + uint col; > + > + /* > + MAX/MIN optimization can convert the subquery into > + expr + Item_singlerow_subselect > + */ (*) > + if (args[1]->type() == Item::SUBSELECT_ITEM) > + sub= (Item_subselect *)args[1]; > + I was looking for the case where the above if statement is not taken. Eventually I have applied this patch, the patch after it (e540d023e2ec6f37efc9ab695ccdfd4a6744ad64) to get the test cases, then added this at line (*): DBUG_ASSERT(args[1]->type() == Item::SUBSELECT_ITEM); and ran the entire main test suite, with and without --ps-protocol. It worked. Can you add test coverage for this case? > if (fix_left(thd, ref)) > return TRUE; > if (args[0]->maybe_null) > @@ -1507,10 +1533,10 @@ bool Item_in_optimizer::fix_fields(THD *thd, Item **ref) > > if (!args[1]->fixed && args[1]->fix_fields(thd, args+1)) > return TRUE; > - Item_in_subselect * sub= (Item_in_subselect *)args[1]; > - if (args[0]->cols() != sub->engine->cols()) > + if ((sub && ((col= args[0]->cols()) != sub->engine->cols())) || > + (!sub && (args[1]->cols() != (col= 1)))) > { > - my_error(ER_OPERAND_COLUMNS, MYF(0), args[0]->cols()); > + my_error(ER_OPERAND_COLUMNS, MYF(0), col); > return TRUE; > } > if (args[1]->maybe_null) ... BR Sergei -- Sergei Petrunia, Software Developer MariaDB Corporation | Skype: sergefp | Blog: http://s.petrunia.net/blog

2 1

Re: [Maria-developers] [Commits] 9f8a458: MDEV-7846: Server crashes in Item_subselect::fix_fields or fails with Thread stack overrun
by Sergey Petrunia 08 Jun '15

08 Jun '15

Hi Sanja, It is very difficult to understand this patch. What is the exact meaning of left_expr_orig? It is declared but has no comments. It is interesting that convert_subq_to_sj has got new code to handle scalar comparison case ( foo IN (SELECT bar ...)), while there is no handling for tuple comparison ( (foo1,foo2) IN (SELECT bar1,bar2 ..)). Is this intentional? On Thu, Apr 23, 2015 at 08:09:00PM +0200, sanja(a)mariadb.com wrote: > revision-id: 9f8a458fb2d07298810bb5d9824ce728017bfb32 > parent(s): e540d023e2ec6f37efc9ab695ccdfd4a6744ad64 > committer: Oleksandr Byelkin > branch nick: server > timestamp: 2015-04-23 20:08:57 +0200 > message: > > MDEV-7846: Server crashes in Item_subselect::fix_fields or fails with Thread stack overrun > > Substitute into transformed subselects original left expression and than register its change in case it was substituted. > > --- > mysql-test/r/subselect.result | 28 +++++++++++++++++++++++++ > mysql-test/r/subselect_no_mat.result | 28 +++++++++++++++++++++++++ > mysql-test/r/subselect_no_opts.result | 28 +++++++++++++++++++++++++ > mysql-test/r/subselect_no_scache.result | 28 +++++++++++++++++++++++++ > mysql-test/r/subselect_no_semijoin.result | 28 +++++++++++++++++++++++++ > mysql-test/t/subselect.test | 34 +++++++++++++++++++++++++++++++ > sql/item_cmpfunc.cc | 8 ++++---- > sql/item_subselect.cc | 8 +++----- > sql/item_subselect.h | 1 + > sql/opt_subselect.cc | 4 +++- > 10 files changed, 185 insertions(+), 10 deletions(-) > > diff --git a/mysql-test/r/subselect.result b/mysql-test/r/subselect.result > index 0ab2d48..cf52ba2 100644 > --- a/mysql-test/r/subselect.result > +++ b/mysql-test/r/subselect.result > @@ -7053,3 +7053,31 @@ WHERE h.host in (SELECT host FROM mysql.user) > ) AS sq > FROM mysql.host h GROUP BY h.host; > sq > +# > +# MDEV-7846:Server crashes in Item_subselect::fix > +#_fields or fails with Thread stack overrun > +# > +CREATE TABLE t1 (column1 INT) ENGINE=MyISAM; > +INSERT INTO t1 VALUES (3),(9); > +CREATE TABLE t2 (column2 INT) ENGINE=MyISAM; > +INSERT INTO t2 VALUES (1),(4); > +CREATE TABLE t3 (column3 INT) ENGINE=MyISAM; > +INSERT INTO t3 VALUES (6),(8); > +CREATE TABLE t4 (column4 INT) ENGINE=MyISAM; > +INSERT INTO t4 VALUES (2),(5); > +PREPARE stmt FROM " > +SELECT ( > + SELECT MAX( table1.column1 ) AS field1 > + FROM t1 AS table1 > + WHERE table3.column3 IN ( SELECT table2.column2 AS field2 FROM t2 AS table2 ) > +) AS sq > +FROM t3 AS table3, t4 AS table4 GROUP BY sq > +"; > +EXECUTE stmt; > +sq > +NULL > +EXECUTE stmt; > +sq > +NULL > +deallocate prepare stmt; > +drop table t1,t2,t3,t4; > diff --git a/mysql-test/r/subselect_no_mat.result b/mysql-test/r/subselect_no_mat.result > index 5780351..73a69e9 100644 > --- a/mysql-test/r/subselect_no_mat.result > +++ b/mysql-test/r/subselect_no_mat.result > @@ -7050,6 +7050,34 @@ WHERE h.host in (SELECT host FROM mysql.user) > ) AS sq > FROM mysql.host h GROUP BY h.host; > sq > +# > +# MDEV-7846:Server crashes in Item_subselect::fix > +#_fields or fails with Thread stack overrun > +# > +CREATE TABLE t1 (column1 INT) ENGINE=MyISAM; > +INSERT INTO t1 VALUES (3),(9); > +CREATE TABLE t2 (column2 INT) ENGINE=MyISAM; > +INSERT INTO t2 VALUES (1),(4); > +CREATE TABLE t3 (column3 INT) ENGINE=MyISAM; > +INSERT INTO t3 VALUES (6),(8); > +CREATE TABLE t4 (column4 INT) ENGINE=MyISAM; > +INSERT INTO t4 VALUES (2),(5); > +PREPARE stmt FROM " > +SELECT ( > + SELECT MAX( table1.column1 ) AS field1 > + FROM t1 AS table1 > + WHERE table3.column3 IN ( SELECT table2.column2 AS field2 FROM t2 AS table2 ) > +) AS sq > +FROM t3 AS table3, t4 AS table4 GROUP BY sq > +"; > +EXECUTE stmt; > +sq > +NULL > +EXECUTE stmt; > +sq > +NULL > +deallocate prepare stmt; > +drop table t1,t2,t3,t4; > set optimizer_switch=default; > select @@optimizer_switch like '%materialization=on%'; > @@optimizer_switch like '%materialization=on%' > diff --git a/mysql-test/r/subselect_no_opts.result b/mysql-test/r/subselect_no_opts.result > index 4bea029..1512e39 100644 > --- a/mysql-test/r/subselect_no_opts.result > +++ b/mysql-test/r/subselect_no_opts.result > @@ -7048,4 +7048,32 @@ WHERE h.host in (SELECT host FROM mysql.user) > ) AS sq > FROM mysql.host h GROUP BY h.host; > sq > +# > +# MDEV-7846:Server crashes in Item_subselect::fix > +#_fields or fails with Thread stack overrun > +# > +CREATE TABLE t1 (column1 INT) ENGINE=MyISAM; > +INSERT INTO t1 VALUES (3),(9); > +CREATE TABLE t2 (column2 INT) ENGINE=MyISAM; > +INSERT INTO t2 VALUES (1),(4); > +CREATE TABLE t3 (column3 INT) ENGINE=MyISAM; > +INSERT INTO t3 VALUES (6),(8); > +CREATE TABLE t4 (column4 INT) ENGINE=MyISAM; > +INSERT INTO t4 VALUES (2),(5); > +PREPARE stmt FROM " > +SELECT ( > + SELECT MAX( table1.column1 ) AS field1 > + FROM t1 AS table1 > + WHERE table3.column3 IN ( SELECT table2.column2 AS field2 FROM t2 AS table2 ) > +) AS sq > +FROM t3 AS table3, t4 AS table4 GROUP BY sq > +"; > +EXECUTE stmt; > +sq > +NULL > +EXECUTE stmt; > +sq > +NULL > +deallocate prepare stmt; > +drop table t1,t2,t3,t4; > set @optimizer_switch_for_subselect_test=null; > diff --git a/mysql-test/r/subselect_no_scache.result b/mysql-test/r/subselect_no_scache.result > index fdb3b12..26cea1f 100644 > --- a/mysql-test/r/subselect_no_scache.result > +++ b/mysql-test/r/subselect_no_scache.result > @@ -7059,6 +7059,34 @@ WHERE h.host in (SELECT host FROM mysql.user) > ) AS sq > FROM mysql.host h GROUP BY h.host; > sq > +# > +# MDEV-7846:Server crashes in Item_subselect::fix > +#_fields or fails with Thread stack overrun > +# > +CREATE TABLE t1 (column1 INT) ENGINE=MyISAM; > +INSERT INTO t1 VALUES (3),(9); > +CREATE TABLE t2 (column2 INT) ENGINE=MyISAM; > +INSERT INTO t2 VALUES (1),(4); > +CREATE TABLE t3 (column3 INT) ENGINE=MyISAM; > +INSERT INTO t3 VALUES (6),(8); > +CREATE TABLE t4 (column4 INT) ENGINE=MyISAM; > +INSERT INTO t4 VALUES (2),(5); > +PREPARE stmt FROM " > +SELECT ( > + SELECT MAX( table1.column1 ) AS field1 > + FROM t1 AS table1 > + WHERE table3.column3 IN ( SELECT table2.column2 AS field2 FROM t2 AS table2 ) > +) AS sq > +FROM t3 AS table3, t4 AS table4 GROUP BY sq > +"; > +EXECUTE stmt; > +sq > +NULL > +EXECUTE stmt; > +sq > +NULL > +deallocate prepare stmt; > +drop table t1,t2,t3,t4; > set optimizer_switch=default; > select @@optimizer_switch like '%subquery_cache=on%'; > @@optimizer_switch like '%subquery_cache=on%' > diff --git a/mysql-test/r/subselect_no_semijoin.result b/mysql-test/r/subselect_no_semijoin.result > index cb6d35d..4c6f037 100644 > --- a/mysql-test/r/subselect_no_semijoin.result > +++ b/mysql-test/r/subselect_no_semijoin.result > @@ -7048,5 +7048,33 @@ WHERE h.host in (SELECT host FROM mysql.user) > ) AS sq > FROM mysql.host h GROUP BY h.host; > sq > +# > +# MDEV-7846:Server crashes in Item_subselect::fix > +#_fields or fails with Thread stack overrun > +# > +CREATE TABLE t1 (column1 INT) ENGINE=MyISAM; > +INSERT INTO t1 VALUES (3),(9); > +CREATE TABLE t2 (column2 INT) ENGINE=MyISAM; > +INSERT INTO t2 VALUES (1),(4); > +CREATE TABLE t3 (column3 INT) ENGINE=MyISAM; > +INSERT INTO t3 VALUES (6),(8); > +CREATE TABLE t4 (column4 INT) ENGINE=MyISAM; > +INSERT INTO t4 VALUES (2),(5); > +PREPARE stmt FROM " > +SELECT ( > + SELECT MAX( table1.column1 ) AS field1 > + FROM t1 AS table1 > + WHERE table3.column3 IN ( SELECT table2.column2 AS field2 FROM t2 AS table2 ) > +) AS sq > +FROM t3 AS table3, t4 AS table4 GROUP BY sq > +"; > +EXECUTE stmt; > +sq > +NULL > +EXECUTE stmt; > +sq > +NULL > +deallocate prepare stmt; > +drop table t1,t2,t3,t4; > set @optimizer_switch_for_subselect_test=null; > set @join_cache_level_for_subselect_test=NULL; > diff --git a/mysql-test/t/subselect.test b/mysql-test/t/subselect.test > index 3eb056d..00ab48b 100644 > --- a/mysql-test/t/subselect.test > +++ b/mysql-test/t/subselect.test > @@ -5930,3 +5930,37 @@ SELECT > WHERE h.host in (SELECT host FROM mysql.user) > ) AS sq > FROM mysql.host h GROUP BY h.host; > + > + > +--echo # > +--echo # MDEV-7846:Server crashes in Item_subselect::fix > +--echo #_fields or fails with Thread stack overrun > +--echo # > +CREATE TABLE t1 (column1 INT) ENGINE=MyISAM; > +INSERT INTO t1 VALUES (3),(9); > + > +CREATE TABLE t2 (column2 INT) ENGINE=MyISAM; > + > +INSERT INTO t2 VALUES (1),(4); > + > +CREATE TABLE t3 (column3 INT) ENGINE=MyISAM; > +INSERT INTO t3 VALUES (6),(8); > + > +CREATE TABLE t4 (column4 INT) ENGINE=MyISAM; > +INSERT INTO t4 VALUES (2),(5); > + > + > +PREPARE stmt FROM " > +SELECT ( > + SELECT MAX( table1.column1 ) AS field1 > + FROM t1 AS table1 > + WHERE table3.column3 IN ( SELECT table2.column2 AS field2 FROM t2 AS table2 ) > +) AS sq > +FROM t3 AS table3, t4 AS table4 GROUP BY sq > +"; > + > +EXECUTE stmt; > +EXECUTE stmt; > + > +deallocate prepare stmt; > +drop table t1,t2,t3,t4; > diff --git a/sql/item_cmpfunc.cc b/sql/item_cmpfunc.cc > index 51a38f0..29fff8e 100644 > --- a/sql/item_cmpfunc.cc > +++ b/sql/item_cmpfunc.cc > @@ -1455,12 +1455,12 @@ bool Item_in_optimizer::fix_left(THD *thd, Item **ref) > next execution we need to copy args[1]->left_expr again. > */ > ref0= &(((Item_in_subselect *)args[1])->left_expr); > - args[0]= ref0[0]; > } > - if ((!args[0]->fixed && args[0]->fix_fields(thd, ref0)) || > - (!cache && !(cache= Item_cache::get_cache(ref0[0])))) > + if ((!(*ref0)->fixed && (*ref0)->fix_fields(thd, ref0)) || > + (!cache && !(cache= Item_cache::get_cache(*ref0)))) > DBUG_RETURN(1); > - args[0]= ref0[0]; > + if (args[0] != (*ref0)) > + current_thd->change_item_tree(args, (*ref0)); > DBUG_PRINT("info", ("actual fix fields")); > > cache->setup(args[0]); > diff --git a/sql/item_subselect.cc b/sql/item_subselect.cc > index 4c11808..e04f5ff 100644 > --- a/sql/item_subselect.cc > +++ b/sql/item_subselect.cc > @@ -1360,7 +1360,7 @@ Item_in_subselect::Item_in_subselect(Item * left_exp, > upper_item(0) > { > DBUG_ENTER("Item_in_subselect::Item_in_subselect"); > - left_expr= left_exp; > + left_expr_orig= left_expr= left_exp; > func= &eq_creator; > init(select_lex, new select_exists_subselect(this)); > max_columns= UINT_MAX; > @@ -1384,7 +1384,7 @@ Item_allany_subselect::Item_allany_subselect(Item * left_exp, > :Item_in_subselect(), func_creator(fc), all(all_arg) > { > DBUG_ENTER("Item_allany_subselect::Item_allany_subselect"); > - left_expr= left_exp; > + left_expr_orig= left_expr= left_exp; > func= func_creator(all_arg); > init(select_lex, new select_exists_subselect(this)); > max_columns= 1; > @@ -2584,15 +2584,13 @@ Item_in_subselect::select_in_like_transformer(JOIN *join) > arena= thd->activate_stmt_arena_if_needed(&backup); > if (!optimizer) > { > - result= (!(optimizer= new Item_in_optimizer(left_expr, this))); > + result= (!(optimizer= new Item_in_optimizer(left_expr_orig, this))); > if (result) > goto out; > } > > thd->lex->current_select= current->return_after_parsing(); > result= optimizer->fix_left(thd, optimizer->arguments()); > - /* fix_fields can change reference to left_expr, we need reassign it */ > - left_expr= optimizer->arguments()[0]; > thd->lex->current_select= current; > > if (changed) > diff --git a/sql/item_subselect.h b/sql/item_subselect.h > index 592e771..930bd66 100644 > --- a/sql/item_subselect.h > +++ b/sql/item_subselect.h > @@ -449,6 +449,7 @@ class Item_in_subselect :public Item_exists_subselect > Item **having_item); > public: > Item *left_expr; > + Item *left_expr_orig; > /* Priority of this predicate in the convert-to-semi-join-nest process. */ > int sj_convert_priority; > /* > diff --git a/sql/opt_subselect.cc b/sql/opt_subselect.cc > index 5b1a7f2..1363be0 100644 > --- a/sql/opt_subselect.cc > +++ b/sql/opt_subselect.cc > @@ -1593,7 +1593,9 @@ static bool convert_subq_to_sj(JOIN *parent_join, Item_in_subselect *subq_pred) > { > nested_join->sj_outer_expr_list.push_back(subq_pred->left_expr); > Item_func_eq *item_eq= > - new Item_func_eq(subq_pred->left_expr, subq_lex->ref_pointer_array[0]); > + new Item_func_eq(subq_pred->left_expr_orig, subq_lex->ref_pointer_array[0]); > + if (subq_pred->left_expr_orig != subq_pred->left_expr) > + thd->change_item_tree(item_eq->arguments(), subq_pred->left_expr); > item_eq->in_equality_no= 0; > sj_nest->sj_on_expr= and_items(sj_nest->sj_on_expr, item_eq); > } > _______________________________________________ > commits mailing list > commits(a)mariadb.org > https://lists.askmonty.org/cgi-bin/mailman/listinfo/commits -- BR Sergei -- Sergei Petrunia, Software Developer MariaDB Corporation | Skype: sergefp | Blog: http://s.petrunia.net/blog

2 1

[Maria-developers] [GSoC] MXS-2
by sriram patil 08 Jun '15

08 Jun '15

Hi All, There is no blog post this week. Could not conclude the task at hand. Will be writing a blog post for 2nd and 3rd week together. Thanks, Sriram

1 0

[Maria-developers] 5.5.44 Sprint: MDEV-8067 correct fix for MySQL Bug # 19699237: UNINITIALIZED VARIABLE IN ITEM_FIELD::STR_RESULT
by Alexander Barkov 08 Jun '15

08 Jun '15

Hi Sergei, Please review my patch for MDEV-8067. Some subselect_(mat|sj) test chanted "EXPLAIN" output. Perhaps the new results are correct. Please advise. Thanks.

1 0

[Maria-developers] Please review MDEV-7269 mysqlbinlog Don't know how to handle column type=0 meta=0 (0000)#
by Alexander Barkov 05 Jun '15

05 Jun '15

Hi Sergei, Please review a joint patch for: MDEV-7269 mysqlbinlog Don't know how to handle column type=0 meta=0 (0000)# MDEV-8267 Add /*old*/ comment into I_S.COLUMN_TYPE for old DECIMAL Thanks.

2 4

[Maria-developers] [maria-developers] MMC Snapin and WMI Provider
by Rolain Djeumen 04 Jun '15

04 Jun '15

Hello to all , I would like to work on the MariaDB MMC Snapin and WMI Provider. Is there anyone in charge of the said feature ?

2 1

Re: [Maria-developers] 91259f9: MDEV-7505 - Too large scale in DECIMAL dynamic column getter crashes mysqld
by Sergei Golubchik 04 Jun '15

04 Jun '15

Hi, svoj! On Jun 04, svoj(a)mariadb.org wrote: > revision-id: 91259f903c2ffac7c7c8fc72385a572eace4fa89 > parent(s): 9bb9bde42dbf6f936faadeaa34cde838bf6b63dd > committer: Sergey Vojtovich > branch nick: mariadb > timestamp: 2015-06-04 16:04:05 +0400 > message: > > MDEV-7505 - Too large scale in DECIMAL dynamic column getter crashes mysqld > > Server may crash if sanity checks of COLUMN_GET() fail. > > COLUMN_GET() description generator expects parent CAST item, which may not have > been created due to failure of sanity checks. Then further attempt to report > an error may crash the server. > > Fixed COLUMN_GET() description generator to handle such case. > > --- > mysql-test/r/dyncol.result | 6 ++++++ > mysql-test/t/AAA.test | 1 + Mistakenly committed file ^^^^ > mysql-test/t/dyncol.test | 6 ++++++ > sql/item_strfunc.cc | 7 +++++++ > 4 files changed, 20 insertions(+) > > diff --git a/mysql-test/t/AAA.test b/mysql-test/t/AAA.test > new file mode 100644 > index 0000000..371db62 > --- /dev/null > +++ b/mysql-test/t/AAA.test > @@ -0,0 +1 @@ > +SELECT COLUMN_GET(`x`, 'y' AS DECIMAL(5,34)); > diff --git a/sql/item_strfunc.cc b/sql/item_strfunc.cc > index eedf149..7ad4c06 100644 > --- a/sql/item_strfunc.cc > +++ b/sql/item_strfunc.cc > @@ -4467,6 +4467,13 @@ bool Item_dyncol_get::get_date(MYSQL_TIME *ltime, ulonglong fuzzy_date) > > void Item_dyncol_get::print(String *str, enum_query_type query_type) How do you get to Item_dyncol_get::print if cast couldn't be created? The comment below says "see create_func_dyncol_get". This function looks like if (!(res= new (thd->mem_root) Item_dyncol_get(str, num))) return res; // Return NULL return create_func_cast(thd, res, cast_type, c_len, c_dec, cs); and to get Item_dyncol_get without a parent, one needs create_func_cast to return 'res' without wrapping it in a CAST(...). But it never does that. > { > + /* Parent cast doesn't exist yet. Only print dynamic column name. */ > + if (!str->length()) Not safe, if you wrap your COLUMN_GET() in, say, CONCAT(): CONCAT(COLUMN_GET(`x`, 'y' AS DECIMAL(5,34))) then str->length() won't be 0, despite the error in CAST. > + { > + args[1]->print(str, query_type); > + return; > + } > + > /* see create_func_dyncol_get */ > DBUG_ASSERT(str->length() >= 5); > DBUG_ASSERT(strncmp(str->ptr() + str->length() - 5, "cast(", 5) == 0); Regards, Sergei

2 2

[Maria-developers] Probably in a bug in simple_pred()
by Alexander Barkov 04 Jun '15

04 Jun '15

Hi Sergey, I probably found a bug in simple_pred() in opt_sum.cc. In this piece of code: > case 3: > /* field BETWEEN const AND const */ > item= func_item->arguments()[0]->real_item(); > if (item->type() == Item::FIELD_ITEM) > { > args[0]= item; > for (int i= 1 ; i <= 2; i++) > { > item= func_item->arguments()[i]->real_item(); > if (!item->const_item()) > return 0; > args[i]= item; > if (check_item1_shorter_item2(args[0], args[1])) Should't it be if (check_item1_shorter_item2(args[0], args[i])) ? > return 0; > } > } If this is a bug, can you please help to make an SQL scrip which would be affected by this bug? Thanks.

2 3