- developers - lists.mariadb.org

Re: [Maria-developers] d1a7ae2: MDEV-9618 solaris sparc build fails on 10.1.
by Sergei Golubchik 26 Jun '16

26 Jun '16

Hi, Alexey! On Jun 26, Alexey Botchkov wrote: > revision-id: d1a7ae232ef6e35e4f81871e3c9d0d318a5dcef1 (mariadb-10.1.14-28-gd1a7ae2) > parent(s): 305bbbce6152b0909937659ecba4d51e5b21c094 > committer: Alexey Botchkov > timestamp: 2016-06-26 00:11:30 +0400 > message: > > MDEV-9618 solaris sparc build fails on 10.1. > > Compiler on Sparc is strict about the 'const' modifiers > in function declarations and definitions. Meaning > they should be exactly same. ok to push Interestingly enough, I've just read about it few days ago - that const modifiers do *not* have to be exactly the same, the compiler can be lax about it, because these const modifiers do not mean anything for the caller. So, this strictness could be considered a bug in that Solaris C++ compiler. Regards, Sergei Chief Architect MariaDB and security(a)mariadb.org

1 0

[Maria-developers] When SP DEFINER is empty.
by Alexey Botchkov 25 Jun '16

25 Jun '16

Hi, Sergey, all. I'm in doubts about how this one should be fixed: https://jira.mariadb.org/browse/MDEV-10119 The story goes like this: - mysql_install_db script runs 'mysqld --skip-grant-tables' In this case the 'current_user()' seems to be empty. - so the CREATE PROCEDURE command in the bootstrap creates the procedure with the empty DEFINER. - as a result, the 'SHOW CREATE PROCEDURE' returns query started with 'CREATE DEFINER=`` PROCEDURE...', - that DEFINER=`` gives an error when feed to the server. That can be fixed on any stage. We can do any of these: - set some 'current_user()' to be not empty even with the --skip-grant-tables option - specify some non-empty DEFINER for the CREATE PROCEDURE statement (in both options it's not that clear what user could that be) - fix the SHOW CREATE PROCEDURE statement so it doesn't add the errorneous DEFINER=`` to the query. - make server handling the 'DEFINER=``' with no error. Maybe assigning the 'current_user()' in this case. So, what can You recommend as a fix in this case? Best regards. HF

2 1

Re: [Maria-developers] [Commits] 773ce40: MDEV-10045: Server crashes in Time_and_counter_tracker::incr_loops
by Sergey Petrunia 24 Jun '16

24 Jun '16

Hi Sanja, O the patch fixes the crash. However when I am debugging it, I see the execution to go like this: (gdb) wher #0 JOIN::optimize_inner (this=0x7fff900087d0) at /home/psergey/dev-git/10.1-dbg6/sql/sql_select.cc:1095 #1 0x0000555555a8e5f8 in JOIN::optimize (this=0x7fff900087d0) at /home/psergey/dev-git/10.1-dbg6/sql/sql_select.cc:1036 #2 0x0000555555b277ce in st_select_lex_unit::optimize (this=0x7fff900068a8) at /home/psergey/dev-git/10.1-dbg6/sql/sql_union.cc:747 #3 0x0000555555b279cb in st_select_lex_unit::exec (this=0x7fff900068a8) at /home/psergey/dev-git/10.1-dbg6/sql/sql_union.cc:780 #4 0x0000555555d3855f in subselect_union_engine::exec (this=0x7fff90007970) at /home/psergey/dev-git/10.1-dbg6/sql/item_subselect.cc:3785 #5 0x0000555555d2eb12 in Item_subselect::exec (this=0x7fff9000b028) at /home/psergey/dev-git/10.1-dbg6/sql/item_subselect.cc:684 #6 0x0000555555d3057e in Item_singlerow_subselect::val_str (this=0x7fff9000b028, str=0x7ffff43b3a40) at /home/psergey/dev-git/10.1-dbg6/sql/item_subselect.cc:1295 #7 0x0000555555cbca5e in Item_func_conv_charset::Item_func_conv_charset (this=0x7fff90021aa0, thd=0x555557ffda80, a=0x7fff9000b028, cs=0x555556c18b80, cache_if_const=true) at /home/psergey/dev-git/10.1-dbg6/sql/item_strfunc.h:975 #8 0x0000555555ca1a7b in Item::safe_charset_converter (this=0x7fff9000b028, thd=0x555557ffda80, tocs=0x555556c18b80) at /home/psergey/dev-git/10.1-dbg6/sql/item.cc:1084 #9 0x0000555555ca4452 in Item_func_or_sum::agg_item_set_converter (this=0x7fff90021828, coll=..., fname=0x555556460377 "<", args=0x7fff900218b0, nargs=2, flags=7, item_sep=1) at /home/psergey/dev-git/10.1-dbg6/sql/item.cc:2141 #10 0x0000555555cd4106 in Item_func_or_sum::agg_arg_charsets (this=0x7fff90021828, c=..., items=0x7fff900218b0, nitems=2, flags=7, item_sep=1) at /home/psergey/dev-git/10.1-dbg6/sql/item.h:3714 #11 0x0000555555cd414e in Item_func_or_sum::agg_arg_charsets_for_comparison (this=0x7fff90021828, c=..., items=0x7fff900218b0, nitems=2, item_sep=1) at /home/psergey/dev-git/10.1-dbg6/sql/item.h:3760 #12 0x0000555555cc10ac in Item_func::setup_args_and_comparator (this=0x7fff90021828, thd=0x555557ffda80, cmp=0x7fff900218e0) at /home/psergey/dev-git/10.1-dbg6/sql/item_cmpfunc.cc:509 #13 0x0000555555cc11b5 in Item_bool_rowready_func2::fix_length_and_dec (this=0x7fff90021828) at /home/psergey/dev-git/10.1-dbg6/sql/item_cmpfunc.cc:531 #14 0x0000555555cf022b in Item_func::fix_fields (this=0x7fff90021828, thd=0x555557ffda80, ref=0x7ffff43b3eb0) at /home/psergey/dev-git/10.1-dbg6/sql/item_func.cc:234 #15 0x0000555555d322fb in Item_allany_subselect::transform_into_max_min (this=0x7fff900077a8, join=0x7fff900087d0) at /home/psergey/dev-git/10.1-dbg6/sql/item_subselect.cc:1969 #16 0x0000555555bd557d in JOIN::transform_max_min_subquery (this=0x7fff900087d0) at /home/psergey/dev-git/10.1-dbg6/sql/opt_subselect.cc:901 #17 0x0000555555a8e9c6 in JOIN::optimize_inner (this=0x7fff900087d0) at /home/psergey/dev-git/10.1-dbg6/sql/sql_select.cc:1131 #18 0x0000555555a8e5f8 in JOIN::optimize (this=0x7fff900087d0) at /home/psergey/dev-git/10.1-dbg6/sql/sql_select.cc:1036 #19 0x0000555555a44f98 in st_select_lex::optimize_unflattened_subqueries (this=0x555558001b98, const_only=false) at /home/psergey/dev-git/10.1-dbg6/sql/sql_lex.cc:3760 #20 0x0000555555bdeaee in JOIN::optimize_unflattened_subqueries (this=0x7fff90007ca8) at /home/psergey/dev-git/10.1-dbg6/sql/opt_subselect.cc:5051 #21 0x0000555555a91dd7 in JOIN::optimize_inner (this=0x7fff90007ca8) at /home/psergey/dev-git/10.1-dbg6/sql/sql_select.cc:2043 #22 0x0000555555a8e5f8 in JOIN::optimize (this=0x7fff90007ca8) at /home/psergey/dev-git/10.1-dbg6/sql/sql_select.cc:1036 #23 0x0000555555a96bf2 in mysql_select (thd=0x555557ffda80, rref_pointer_array=0x555558001e10, tables=0x7fff900054b0, wild_num=1, fields=..., conds=0x7fff90007a80, og_num=0, order=0x0, group=0x0, having=0x0, proc_param=0x0, select_options=2147748608, result=0x7fff90007c88, unit=0x555558001498, select_lex=0x555558001b98) at /home/psergey/dev-git/10.1-dbg6/sql/sql_select.cc:3437 #24 0x0000555555a8c7fd in handle_select (thd=0x555557ffda80, lex=0x5555580013d0, result=0x7fff90007c88, setup_tables_done_option=0) at /home/psergey/dev-git/10.1-dbg6/sql/sql_select.cc:384 #25 0x0000555555a5cd61 in execute_sqlcom_select (thd=0x555557ffda80, all_tables=0x7fff900054b0) at /home/psergey/dev-git/10.1-dbg6/sql/sql_parse.cc:5894 #26 0x0000555555a52a21 in mysql_execute_command (thd=0x555557ffda80) at /home/psergey/dev-git/10.1-dbg6/sql/sql_parse.cc:2960 #27 0x0000555555a604b1 in mysql_parse (thd=0x555557ffda80, rawbuf=0x7fff90005258 "SELECT * FROM t1, t2 WHERE f3 = f2 AND f1 > ANY ( SELECT 'foo' UNION SELECT 'bar' )", length=83, parser_state=0x7ffff43b54f0) at /home/psergey/dev-git/10.1-dbg6/sql/sql_parse.cc:7314 #28 0x0000555555a4ec23 in dispatch_command (command=COM_QUERY, thd=0x555557ffda80, packet=0x555558004031 "SELECT * FROM t1, t2 WHERE f3 = f2 AND f1 > ANY ( SELECT 'foo' UNION SELECT 'bar' )", packet_length=83) at /home/psergey/dev-git/10.1-dbg6/sql/sql_parse.cc:1486 #29 0x0000555555a4d944 in do_command (thd=0x555557ffda80) at /home/psergey/dev-git/10.1-dbg6/sql/sql_parse.cc:1107 Note frame #0 and frame #17: JOIN::optimize_inner() has invoked a few functions which in turn invoked JOIN::optimize (and JOIN::optimize_inner()) for the same join. Before your patch this didn't happen, as the inner call to JOIN::optimize would find join->optimized= true and return immediately. In this particular example, the double-optimized join a degenerate "SELECT 'foo'", double-optimization has no effect. I tried to create an example with a non-degenerate join but didn't succeed. Looking at the code, I have no certainity that it's impossible to have two JOIN::optimize calls nest for a non-degenerate join. If you can explain why this is not possible, please do. If you can't prove this, I would suggest adding JOIN::inside_optimize_call and have "if (inside_optimize_call) return 0;" as the first line of JOIN::optimize. On Wed, Jun 22, 2016 at 11:17:44AM +0200, Oleksandr Byelkin wrote: > revision-id: 773ce408762b5f8256d4053b6d0d418d15657b92 (mariadb-10.1.14-24-g773ce40) > parent(s): 63120090f994cc78876944e9f7a76f53337fa46e > committer: Oleksandr Byelkin > timestamp: 2016-06-22 11:17:44 +0200 > message: > > MDEV-10045: Server crashes in Time_and_counter_tracker::incr_loops > > Do not set 'optimized' flag until whole optimization procedure is finished. > > --- > mysql-test/r/subselect.result | 11 +++++++++++ > mysql-test/r/subselect_no_exists_to_in.result | 11 +++++++++++ > mysql-test/r/subselect_no_mat.result | 11 +++++++++++ > mysql-test/r/subselect_no_opts.result | 11 +++++++++++ > mysql-test/r/subselect_no_scache.result | 11 +++++++++++ > mysql-test/r/subselect_no_semijoin.result | 11 +++++++++++ > mysql-test/t/subselect.test | 15 +++++++++++++++ > sql/sql_select.cc | 6 +++--- > 8 files changed, 84 insertions(+), 3 deletions(-) > > diff --git a/mysql-test/r/subselect.result b/mysql-test/r/subselect.result > index 6a53199..1ea7000 100644 > --- a/mysql-test/r/subselect.result > +++ b/mysql-test/r/subselect.result > @@ -7159,3 +7159,14 @@ f > foo > drop table t1; > SET NAMES default; > +# > +# MDEV-10045: Server crashes in Time_and_counter_tracker::incr_loops > +# > +SET NAMES utf8; > +CREATE TABLE t1 (f1 VARCHAR(3), f2 INT UNSIGNED) ENGINE=MyISAM; > +CREATE TABLE t2 (f3 INT) ENGINE=MyISAM; > +SELECT * FROM t1, t2 WHERE f3 = f2 AND f1 > ANY ( SELECT 'foo' UNION SELECT 'bar' ); > +f1 f2 f3 > +DROP TABLE t1, t2; > +SET NAMES default; > +End of 10.1 tests > diff --git a/mysql-test/r/subselect_no_exists_to_in.result b/mysql-test/r/subselect_no_exists_to_in.result > index aa68434..be5e5b6 100644 > --- a/mysql-test/r/subselect_no_exists_to_in.result > +++ b/mysql-test/r/subselect_no_exists_to_in.result > @@ -7159,6 +7159,17 @@ f > foo > drop table t1; > SET NAMES default; > +# > +# MDEV-10045: Server crashes in Time_and_counter_tracker::incr_loops > +# > +SET NAMES utf8; > +CREATE TABLE t1 (f1 VARCHAR(3), f2 INT UNSIGNED) ENGINE=MyISAM; > +CREATE TABLE t2 (f3 INT) ENGINE=MyISAM; > +SELECT * FROM t1, t2 WHERE f3 = f2 AND f1 > ANY ( SELECT 'foo' UNION SELECT 'bar' ); > +f1 f2 f3 > +DROP TABLE t1, t2; > +SET NAMES default; > +End of 10.1 tests > set optimizer_switch=default; > select @@optimizer_switch like '%exists_to_in=off%'; > @@optimizer_switch like '%exists_to_in=off%' > diff --git a/mysql-test/r/subselect_no_mat.result b/mysql-test/r/subselect_no_mat.result > index 754aec1..6dd227c 100644 > --- a/mysql-test/r/subselect_no_mat.result > +++ b/mysql-test/r/subselect_no_mat.result > @@ -7152,6 +7152,17 @@ f > foo > drop table t1; > SET NAMES default; > +# > +# MDEV-10045: Server crashes in Time_and_counter_tracker::incr_loops > +# > +SET NAMES utf8; > +CREATE TABLE t1 (f1 VARCHAR(3), f2 INT UNSIGNED) ENGINE=MyISAM; > +CREATE TABLE t2 (f3 INT) ENGINE=MyISAM; > +SELECT * FROM t1, t2 WHERE f3 = f2 AND f1 > ANY ( SELECT 'foo' UNION SELECT 'bar' ); > +f1 f2 f3 > +DROP TABLE t1, t2; > +SET NAMES default; > +End of 10.1 tests > 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 e05dd4d..389c0f5 100644 > --- a/mysql-test/r/subselect_no_opts.result > +++ b/mysql-test/r/subselect_no_opts.result > @@ -7150,4 +7150,15 @@ f > foo > drop table t1; > SET NAMES default; > +# > +# MDEV-10045: Server crashes in Time_and_counter_tracker::incr_loops > +# > +SET NAMES utf8; > +CREATE TABLE t1 (f1 VARCHAR(3), f2 INT UNSIGNED) ENGINE=MyISAM; > +CREATE TABLE t2 (f3 INT) ENGINE=MyISAM; > +SELECT * FROM t1, t2 WHERE f3 = f2 AND f1 > ANY ( SELECT 'foo' UNION SELECT 'bar' ); > +f1 f2 f3 > +DROP TABLE t1, t2; > +SET NAMES default; > +End of 10.1 tests > 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 71ade62..a90278c 100644 > --- a/mysql-test/r/subselect_no_scache.result > +++ b/mysql-test/r/subselect_no_scache.result > @@ -7165,6 +7165,17 @@ f > foo > drop table t1; > SET NAMES default; > +# > +# MDEV-10045: Server crashes in Time_and_counter_tracker::incr_loops > +# > +SET NAMES utf8; > +CREATE TABLE t1 (f1 VARCHAR(3), f2 INT UNSIGNED) ENGINE=MyISAM; > +CREATE TABLE t2 (f3 INT) ENGINE=MyISAM; > +SELECT * FROM t1, t2 WHERE f3 = f2 AND f1 > ANY ( SELECT 'foo' UNION SELECT 'bar' ); > +f1 f2 f3 > +DROP TABLE t1, t2; > +SET NAMES default; > +End of 10.1 tests > 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 43d191b..16aa7b8 100644 > --- a/mysql-test/r/subselect_no_semijoin.result > +++ b/mysql-test/r/subselect_no_semijoin.result > @@ -7150,5 +7150,16 @@ f > foo > drop table t1; > SET NAMES default; > +# > +# MDEV-10045: Server crashes in Time_and_counter_tracker::incr_loops > +# > +SET NAMES utf8; > +CREATE TABLE t1 (f1 VARCHAR(3), f2 INT UNSIGNED) ENGINE=MyISAM; > +CREATE TABLE t2 (f3 INT) ENGINE=MyISAM; > +SELECT * FROM t1, t2 WHERE f3 = f2 AND f1 > ANY ( SELECT 'foo' UNION SELECT 'bar' ); > +f1 f2 f3 > +DROP TABLE t1, t2; > +SET NAMES default; > +End of 10.1 tests > 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 3599b52..c9c89a2 100644 > --- a/mysql-test/t/subselect.test > +++ b/mysql-test/t/subselect.test > @@ -6011,3 +6011,18 @@ INSERT INTO t1 VALUES ('foo'); > SELECT f FROM t1 WHERE f > ALL ( SELECT 'bar' UNION SELECT 'baz' ); > drop table t1; > SET NAMES default; > + > +--echo # > +--echo # MDEV-10045: Server crashes in Time_and_counter_tracker::incr_loops > +--echo # > +SET NAMES utf8; > + > +CREATE TABLE t1 (f1 VARCHAR(3), f2 INT UNSIGNED) ENGINE=MyISAM; > +CREATE TABLE t2 (f3 INT) ENGINE=MyISAM; > + > +SELECT * FROM t1, t2 WHERE f3 = f2 AND f1 > ANY ( SELECT 'foo' UNION SELECT 'bar' ); > + > +DROP TABLE t1, t2; > +SET NAMES default; > + > +--echo End of 10.1 tests > diff --git a/sql/sql_select.cc b/sql/sql_select.cc > index 4825726..ded59eb 100644 > --- a/sql/sql_select.cc > +++ b/sql/sql_select.cc > @@ -1040,7 +1040,7 @@ int JOIN::optimize() > and deleted. The second call will not produce a valid query plan, it will > short-circuit because optimized==TRUE. > > - "was_optimized != optimized" is here to handle this case: > + "!was_optimized" is here to handle this case: > - first optimization starts, gets an error (from a const. cheap > subquery), returns 1 > - another JOIN::optimize() call made, and now join->optimize() will > @@ -1049,7 +1049,7 @@ int JOIN::optimize() > Can have QEP_NOT_PRESENT_YET for degenerate queries (for example, > SELECT * FROM tbl LIMIT 0) > */ > - if (was_optimized != optimized && !res && have_query_plan != QEP_DELETED) > + if (!was_optimized && !res && have_query_plan != QEP_DELETED) > { > create_explain_query_if_not_exists(thd->lex, thd->mem_root); > have_query_plan= QEP_AVAILABLE; > @@ -1058,6 +1058,7 @@ int JOIN::optimize() > !skip_sort_order && !no_order && (order || group_list), > select_distinct); > } > + optimized= 1; > return res; > } > > @@ -1086,7 +1087,6 @@ JOIN::optimize_inner() > // to prevent double initialization on EXPLAIN > if (optimized) > DBUG_RETURN(0); > - optimized= 1; > DEBUG_SYNC(thd, "before_join_optimize"); > > THD_STAGE_INFO(thd, stage_optimizing); > _______________________________________________ > 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

1 0

Re: [Maria-developers] [Commits] 475cab8: MDEV-10050: Crash in subselect
by Sergey Petrunia 24 Jun '16

24 Jun '16

Hi Sanja, On Wed, Jun 22, 2016 at 02:17:06PM +0200, Oleksandr Byelkin wrote: > revision-id: 475cab835fb48c91d5cca649ab93917ec1718d75 (mariadb-5.5.50-6-g475cab8) > parent(s): a482e76e65a4fee70479e877929381c86b1ec62f > committer: Oleksandr Byelkin > timestamp: 2016-06-22 14:17:06 +0200 > message: > > MDEV-10050: Crash in subselect > > thd should not be taken earlier then fix_field and reset on fix_fields if it is needed. > > --- > sql/item_subselect.cc | 15 ++++++++++++--- > 1 file changed, 12 insertions(+), 3 deletions(-) > > diff --git a/sql/item_subselect.cc b/sql/item_subselect.cc > index ba67474..60cdd3f 100644 > --- a/sql/item_subselect.cc > +++ b/sql/item_subselect.cc > @@ -79,7 +79,9 @@ void Item_subselect::init(st_select_lex *select_lex, > DBUG_PRINT("enter", ("select_lex: 0x%lx this: 0x%lx", > (ulong) select_lex, (ulong) this)); > unit= select_lex->master_unit(); > - thd= unit->thd; > +#ifndef DBUG_OFF > + thd= 0; > +#endif So I've applied the patch, and I'm debugging this statement: prepare s from 'select a, (select max(one_k.a) from one_k where one_k.a <ten.a) from ten'; (this is just the first subquery I tried, nothing special about it). We arrive at the above #ifndef, and I have thd=0xa5a5a5a5a5, that is, it's uninitialized data. I let it execute further... > if (unit->item) > { > /* > Item can be changed in JOIN::prepare while engine in JOIN::optimize > => we do not copy old_engine here > */ > engine= unit->item->engine; > own_engine= FALSE; > parsing_place= unit->item->parsing_place; > unit->thd->change_item_tree((Item**)&unit->item, this); > engine->change_result(this, result, TRUE); > } > else > { > SELECT_LEX *outer_select= unit->outer_select(); > /* > do not take into account expression inside aggregate functions because > they can access original table fields > */ > parsing_place= (outer_select->in_sum_expr ? > NO_MATTER : > outer_select->parsing_place); > if (unit->is_union()) > engine= new subselect_union_engine(thd, unit, result, this); > else > engine= new subselect_single_select_engine(thd, select_lex, result, this); and it arrives here. Stepping into this call, I eventually reach where this stack trace shows: (gdb) wher #0 select_result::set_thd (this=0x7fffb002c130, thd_arg=0x0) at /home/psergey/dev-git/5.5/sql/sql_class.h:3316 #1 0x000000000086fd30 in subselect_engine::set_thd (this=0x7fffb002c150, thd_arg=0x0) at /home/psergey/dev-git/5.5/sql/item_subselect.cc:2892 #2 0x00000000007648e9 in subselect_engine::subselect_engine (this=0x7fffb002c150, thd_arg=0x0, si=0x7fffb002c010, res=0x7fffb002c130) at /home/psergey/dev-git/5.5/sql/item_subselect.h:726 #3 0x000000000086fd66 in subselect_single_select_engine::subselect_single_select_engine (this=0x7fffb002c150, thd_arg=0x0, select=0x7fffb002a8e0, result_arg=0x7fffb002c130, item_arg=0x7fffb002c010) at /home/psergey/dev-git/5.5/sql/item_subselect.cc:2902 #4 0x0000000000868232 in Item_subselect::init (this=0x7fffb002c010, select_lex=0x7fffb002a8e0, result=0x7fffb002c130) at /home/psergey/dev-git/5.5/sql/item_subselect.cc:111 #5 0x000000000086a0d1 in Item_singlerow_subselect::Item_singlerow_subselect (this=0x7fffb002c010, select_lex=0x7fffb002a8e0) at /home/psergey/dev-git/5.5/sql/item_subselect.cc:906 #6 0x000000000077d36f in MYSQLparse (thd=0x2fd99e0) at /home/psergey/dev-git/5.5/sql/sql_yacc.yy:8366 #7 0x00000000006214f2 in parse_sql (thd=0x2fd99e0, parser_state=0x7ffff7f81800, creation_ctx=0x0) at /home/psergey/dev-git/5.5/sql/sql_parse.cc:7818 #8 0x0000000000633dda in Prepared_statement::prepare (this=0x7fffb00167f0, packet=0x7fffb00063f8 "select a, (select max(one_k.a) from one_k where one_k.a <ten.a) from ten", packet_len=72) at /home/psergey/dev-git/5.5/sql/sql_prepare.cc:3353 #9 0x0000000000631a89 in mysql_sql_stmt_prepare (thd=0x2fd99e0) at /home/psergey/dev-git/5.5/sql/sql_prepare.cc:2458 #10 0x000000000061324d in mysql_execute_command (thd=0x2fd99e0) at /home/psergey/dev-git/5.5/sql/sql_parse.cc:2239 #11 0x000000000061d58d in mysql_parse (thd=0x2fd99e0, rawbuf=0x7fffb00062c8 "prepare s from 'select a, (select max(one_k.a) from one_k where one_k.a <ten.a) from ten'", length=89, parser_state=0x7ffff7f82540) at /home/psergey/dev-git/5.5/sql/sql_parse.cc:5934 Note that thd_arg is passing around un-initialized data all of this time. Is this how it is intended to work? This looks scary, a lot of functions are passed garbage as parameter. I assume the test suite passed with this patch, so these function don't do anything with the parameter (they probably save away the THD pointer, and then somebody assigns the pointer a valid value before it is used). Still, this looks really scary, we need to either fix this (strongly preferable), or add a lot of comments about it (acceptable as a last resort measure). > if (unit->item) > { > @@ -90,7 +92,7 @@ void Item_subselect::init(st_select_lex *select_lex, > engine= unit->item->engine; > own_engine= FALSE; > parsing_place= unit->item->parsing_place; > - thd->change_item_tree((Item**)&unit->item, this); > + unit->thd->change_item_tree((Item**)&unit->item, this); > engine->change_result(this, result, TRUE); > } > else > @@ -220,6 +222,10 @@ bool Item_subselect::fix_fields(THD *thd_param, Item **ref) > uint8 uncacheable; > bool res; > > + thd= thd_param; > + > + DBUG_ASSERT(unit->thd == thd); > + > status_var_increment(thd_param->status_var.feature_subquery); > > DBUG_ASSERT(fixed == 0); > @@ -2651,7 +2657,10 @@ bool Item_in_subselect::fix_fields(THD *thd_arg, Item **ref) > { > uint outer_cols_num; > List<Item> *inner_cols; > - char const *save_where= thd->where; > + char const *save_where= thd_arg->where; > + > + thd= thd_arg; > + DBUG_ASSERT(unit->thd == thd); > > if (test_strategy(SUBS_SEMI_JOIN)) > return !( (*ref)= new Item_int(1)); > _______________________________________________ > 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] Executing query inside mysql server
by knizhnik 24 Jun '16

24 Jun '16

Hi, I wonder what is the simplest way to execute SQL query inside server? In PostgreSQL there is SPI API, SQLite allows to use client API,... but I failed to find right way of running queries inside mySQL server. My task is the following: I want to patch mySQL server by redirecting execution of some queries to some external executor. The text of this queries will be stored inside database in some special table. So when some query is executed by mySQL server, I want to perform lookup in this table. What is the simplest way to do it in mySQL?

2 1

[Maria-developers] Why/how does the MariaDB client depend on libdbd-mysql-perl?
by Otto Kekäläinen 24 Jun '16

24 Jun '16

I am returning to this age-old discussion about libdbd-mysql-perl as dependency in Debian: 2015-07-21 19:50 GMT+03:00 Sergei Golubchik <vuvova(a)gmail.com>: >> + libdbi-perl, >> + libterm-readkey-perl, >> + mariadb-client-core-10.0 (>= ${source:Version}), >> + ${misc:Depends}, >> + ${perl:Depends}, >> + ${shlibs:Depends} >> +Recommends: libdbd-mysql-perl (>= 1.2202) > > really? why libdbi-perl is required but libdbd-perl is optional? > the one makes no sense without the other. and libterm-readkey-perl > is required, why it's really optional, everything works without it. In MySQL 5.6 this dependency also still exists: http://anonscm.debian.org/cgit/pkg-mysql/mysql-5.6.git/tree/debian/control Package: mysql-client-5.6 Architecture: any Depends: debianutils (>=1.6), libdbd-mysql-perl (>= 1.2202), libdbi-perl, libterm-readkey-perl, But it has been dropped in MySQL 5.7 packaging. In 5.7 it is not even a Recommends or Suggests, but completely removed. http://anonscm.debian.org/cgit/pkg-mysql/mysql-5.7.git/tree/debian/control I don't know what to do about this because I don't really understand what or how the mysql client does with perl library dependencies. It is a C program, does it really use the perl library in any way? So auxilary script does something? Can we drop the dependency on libdbd-mysql-perl, and starting from what version? - Otto

2 1

[Maria-developers] MDEV-8502 DECIMAL accepts out of range DEFAULT values
by Alexander Barkov 24 Jun '16

24 Jun '16

Hello Sergei, Please review a patch fixing: MDEV-8502 DECIMAL accepts out of range DEFAULT values MDEV-10277 Redundant NOTE when inserting '0.00001 ' into a DECIMAL(2,1) column Thanks.

1 0

Re: [Maria-developers] [MariaDB/server] MDEV-10271: Log stop of slave SQL thread on errors, too (#188)
by Kristian Nielsen 23 Jun '16

23 Jun '16

Hartmut Holzgraefe <notifications(a)github.com> writes: > -- Commit Summary -- > > * MDEV-10271: Log stop of slave SQL thread on errors, too > * MDEV-10272: add master host/port info to slave thread exit messages So one comment. With the first patch, there are two cases where the error message "Slave SQL thread exiting, replication stopped ..." will be logged even though the thread fails before it gets to log the corresponding start message "Slave SQL thread initialized, starting replication ..." Was this intended, it seems perhaps a bit confusing if the log has spurious messages about threads stopping, when there are no corresponding start messages? If not intended, it is simple to change by adding an extra label err_after_start: before the message, and using that or err: as appropriate in the gotos. What do you think? But I see you filed pull requests against 10.0 and 10.1, so it seems you were intending these for the stable releases? I do not have any strong opinion on this myself, but for stable releases, you should write an email to maria-developers(a)lists.launchpad.net to solicit opinions about making an exception for this case. Include justification of why it should go in stable releases (and not just in 10.2), and discussion of possible impact on existing applications/setups. If there are no objections, I'll push them. - Kristian.

1 0

[Maria-developers] Formatting rules for double => varchar assignment
by Jesse Schalken 23 Jun '16

23 Jun '16

Hi MariaDB Developers, I have an application which depends on the formatting MariaDB applies to a double when assigned to a char/varchar column of different lengths, and am trying to reproduce the behaviour in PHP. I have gotten some way but just can't seem to get the rules right. I'm having trouble finding a specification for this in the docs or the code that is responsible for this in the MariaDB source. Can someone point me in the right direction? Thanks

2 4

Re: [Maria-developers] GSoC weekly reports (Unique indexes for blobs)
by Shubham Barai 22 Jun '16

22 Jun '16

GSoC (week 4) Hello everyone, After working on create table operation,next thing I had to work on was insert operations.So I explored some of the functions like row_ins_scan_index_for_duplicates, btr_pcur_get_rec to get clear understanding about how to implement duplicate search on hash index. There was a problem in hash function that I wrote .It would calculate same hash value for two different keys if the prefix length of blob key part was zero. Now it seems to be working after I checked it in debugger.I still have to modify it for data types like varchar etc. I have added test cases for insert operations in myisam. In MyIsam, I found one problem in update operation. When updating a row,if the key is conflicting then server crashes because some pointer goes invalid in compare_record. I haven't fixed this issue yet. I also modified some functions in dict0load.cc to adjust some members of dict_index_t for a new index type.The main problem is that index entry for hash based index cointains only two fields(hash value and row id) while dict_index_t contains hash field and other user defined fields which are used to calculate hash value.Some of the operations like alter table( e.g. rename column) needs to get access to all fields while other functions like rec_get_offsets and row_build_index_entry_low needs to get access to only hash field and row id. I am still working on this to find efficient solution to this problem. On 16 June 2016 at 23:29, Sergei Golubchik <vuvova(a)gmail.com> wrote: > Hi, Shubham! > > What I wanted to say on IRC was: > > here's what the comment of cmp_dtuple_rec_with_match_low() says: > > ............... If rec has an externally stored field we do not > compare it but return with value 0 if such a comparison should be > made. > > Note that blobs are externally stored fields in InnoDB, so, I think, > this means that you cannot use cmp_dtuple_rec() to compare blobs. > > Regards, > Sergei > Chief Architect MariaDB > and security(a)mariadb.org >

2 1