diff options
| author | unknown <sanja@askmonty.org> | 2012-12-28 14:41:46 +0200 |
|---|---|---|
| committer | unknown <sanja@askmonty.org> | 2012-12-28 14:41:46 +0200 |
| commit | 6f26aac9409e3456798e58a4ee4306e43c7ebf7b (patch) | |
| tree | df8ad17736247d1493460205865fb75757a00ae1 /sql/item_xmlfunc.h | |
| parent | e99aa91e90adfd54cc1f460dd8cdd19614f30abc (diff) | |
| download | mariadb-git-6f26aac9409e3456798e58a4ee4306e43c7ebf7b.tar.gz | |
MDEV-3873 & MDEV-3876 & MDEV-3912 : Wrong result (extra rows) with ALL subquery
from a MERGE view.
The problem was in the lost ability to be null for the table of a left join if it
is a view/derived table.
It hapenned because setup_table_map(), was called earlier then we merged
the view or derived.
Fixed by propagating new maybe_null flag during Item::update_used_tables().
Change in join_outer.test and join_outer_jcl6.test appeared because
IS NULL reported no used tables (i.e. constant) for argument which could not be
NULL and new maybe_null flag was propagated for IS NULL argument (Item_field)
because table the Item_field belonged to changed its maybe_null status.
Diffstat (limited to 'sql/item_xmlfunc.h')
| -rw-r--r-- | sql/item_xmlfunc.h | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/sql/item_xmlfunc.h b/sql/item_xmlfunc.h index ce33d161c79..be56a83349f 100644 --- a/sql/item_xmlfunc.h +++ b/sql/item_xmlfunc.h @@ -30,14 +30,10 @@ protected: public: Item_xml_str_func(Item *a, Item *b): Item_str_func(a,b) - { - maybe_null= TRUE; - } + {} Item_xml_str_func(Item *a, Item *b, Item *c): Item_str_func(a,b,c) - { - maybe_null= TRUE; - } + {} void fix_length_and_dec(); String *parse_xml(String *raw_xml, String *parsed_xml_buf); bool check_vcol_func_processor(uchar *int_arg) |