mirror of
https://github.com/MariaDB/server.git
synced 2025-08-07 00:04:31 +03:00
MDEV-9701: CREATE VIEW with GROUP BY or ORDER BY and constant produces invalid definition
Fixed printing integer constant in the ORDER clause (MySQL solution) Removed workaround for double resolving counter in the ORDER.
This commit is contained in:
@@ -826,6 +826,9 @@ drop table t2;
|
||||
create table t1 (a int);
|
||||
insert into t1 values (1), (2);
|
||||
create view v1 as select 5 from t1 order by 1;
|
||||
show create view v1;
|
||||
View Create View character_set_client collation_connection
|
||||
v1 CREATE ALGORITHM=UNDEFINED DEFINER=`root`@`localhost` SQL SECURITY DEFINER VIEW `v1` AS select 5 AS `5` from `t1` order by 1 latin1 latin1_swedish_ci
|
||||
select * from v1;
|
||||
5
|
||||
5
|
||||
@@ -5884,5 +5887,21 @@ a
|
||||
DROP VIEW v1;
|
||||
DROP TABLE t1;
|
||||
#
|
||||
# MDEV-9701: CREATE VIEW with GROUP BY or ORDER BY and constant
|
||||
# produces invalid definition
|
||||
#
|
||||
CREATE TABLE t1 ( i INT );
|
||||
INSERT INTO t1 VALUES (1),(2);
|
||||
CREATE VIEW v1 AS
|
||||
SELECT 3 AS three, COUNT(*) FROM t1 GROUP BY three;
|
||||
show create view v1;
|
||||
View Create View character_set_client collation_connection
|
||||
v1 CREATE ALGORITHM=UNDEFINED DEFINER=`root`@`localhost` SQL SECURITY DEFINER VIEW `v1` AS select 3 AS `three`,count(0) AS `COUNT(*)` from `t1` group by '' latin1 latin1_swedish_ci
|
||||
SELECT * FROM v1;
|
||||
three COUNT(*)
|
||||
3 2
|
||||
drop view v1;
|
||||
drop table t1;
|
||||
#
|
||||
# End of 10.1 tests
|
||||
#
|
||||
|
@@ -731,6 +731,7 @@ drop table t2;
|
||||
create table t1 (a int);
|
||||
insert into t1 values (1), (2);
|
||||
create view v1 as select 5 from t1 order by 1;
|
||||
show create view v1;
|
||||
select * from v1;
|
||||
drop view v1;
|
||||
drop table t1;
|
||||
@@ -5747,6 +5748,22 @@ SELECT * FROM v1 WHERE a=_latin1'a' COLLATE latin1_bin;
|
||||
DROP VIEW v1;
|
||||
DROP TABLE t1;
|
||||
|
||||
--echo #
|
||||
--echo # MDEV-9701: CREATE VIEW with GROUP BY or ORDER BY and constant
|
||||
--echo # produces invalid definition
|
||||
--echo #
|
||||
CREATE TABLE t1 ( i INT );
|
||||
INSERT INTO t1 VALUES (1),(2);
|
||||
|
||||
CREATE VIEW v1 AS
|
||||
SELECT 3 AS three, COUNT(*) FROM t1 GROUP BY three;
|
||||
|
||||
show create view v1;
|
||||
|
||||
SELECT * FROM v1;
|
||||
|
||||
drop view v1;
|
||||
drop table t1;
|
||||
|
||||
--echo #
|
||||
--echo # End of 10.1 tests
|
||||
|
@@ -2688,8 +2688,6 @@ void st_select_lex::print_order(String *str,
|
||||
for (; order; order= order->next)
|
||||
{
|
||||
if (order->counter_used)
|
||||
{
|
||||
if (!(query_type & QT_VIEW_INTERNAL))
|
||||
{
|
||||
char buffer[20];
|
||||
size_t length= my_snprintf(buffer, 20, "%d", order->counter);
|
||||
@@ -2697,22 +2695,16 @@ void st_select_lex::print_order(String *str,
|
||||
}
|
||||
else
|
||||
{
|
||||
/* replace numeric reference with expression */
|
||||
/* replace numeric reference with equivalent for ORDER constant */
|
||||
if (order->item[0]->type() == Item::INT_ITEM &&
|
||||
order->item[0]->basic_const_item())
|
||||
{
|
||||
char buffer[20];
|
||||
size_t length= my_snprintf(buffer, 20, "%d", order->counter);
|
||||
str->append(buffer, (uint) length);
|
||||
/* make it expression instead of integer constant */
|
||||
str->append(STRING_WITH_LEN("+0"));
|
||||
str->append(STRING_WITH_LEN("''"));
|
||||
}
|
||||
else
|
||||
(*order->item)->print(str, query_type);
|
||||
}
|
||||
}
|
||||
else
|
||||
(*order->item)->print(str, query_type);
|
||||
if (!order->asc)
|
||||
str->append(STRING_WITH_LEN(" desc"));
|
||||
if (order->next)
|
||||
|
@@ -21874,7 +21874,11 @@ find_order_in_list(THD *thd, Item **ref_pointer_array, TABLE_LIST *tables,
|
||||
*/
|
||||
if (order_item->type() == Item::INT_ITEM && order_item->basic_const_item())
|
||||
{ /* Order by position */
|
||||
uint count= (uint) order_item->val_int();
|
||||
uint count;
|
||||
if (order->counter_used)
|
||||
count= order->counter; // counter was once resolved
|
||||
else
|
||||
count= (uint) order_item->val_int();
|
||||
if (!count || count > fields.elements)
|
||||
{
|
||||
my_error(ER_BAD_FIELD_ERROR, MYF(0),
|
||||
|
@@ -1161,11 +1161,22 @@ List<Item> *st_select_lex_unit::get_unit_column_types()
|
||||
return &sl->item_list;
|
||||
}
|
||||
|
||||
|
||||
static void cleanup_order(ORDER *order)
|
||||
{
|
||||
for (; order; order= order->next)
|
||||
order->counter_used= 0;
|
||||
}
|
||||
|
||||
|
||||
bool st_select_lex::cleanup()
|
||||
{
|
||||
bool error= FALSE;
|
||||
DBUG_ENTER("st_select_lex::cleanup()");
|
||||
|
||||
cleanup_order(order_list.first);
|
||||
cleanup_order(group_list.first);
|
||||
|
||||
if (join)
|
||||
{
|
||||
DBUG_ASSERT((st_select_lex*)join->select_lex == this);
|
||||
|
Reference in New Issue
Block a user