Commit aafca9f0 authored by unknown's avatar unknown

An improvement of the fix for the bug #2231: string column,

INDEX+LIKE, don't take the ESCAPE character. (ver. 2)

parent 824111e6
...@@ -42,4 +42,7 @@ insert into t1 values ('a'), ('a\\b'); ...@@ -42,4 +42,7 @@ insert into t1 values ('a'), ('a\\b');
select * from t1 where a like 'a\\%' escape '#'; select * from t1 where a like 'a\\%' escape '#';
a a
a\b a\b
select * from t1 where a like 'a\\%' escape '#' and a like 'a\\\\b';
a
a\b
drop table t1; drop table t1;
...@@ -29,4 +29,5 @@ drop table t1; ...@@ -29,4 +29,5 @@ drop table t1;
create table t1 (a varchar(10), key(a)); create table t1 (a varchar(10), key(a));
insert into t1 values ('a'), ('a\\b'); insert into t1 values ('a'), ('a\\b');
select * from t1 where a like 'a\\%' escape '#'; select * from t1 where a like 'a\\%' escape '#';
select * from t1 where a like 'a\\%' escape '#' and a like 'a\\\\b';
drop table t1; drop table t1;
...@@ -290,7 +290,7 @@ typedef struct st_qsel_param { ...@@ -290,7 +290,7 @@ typedef struct st_qsel_param {
char min_key[MAX_KEY_LENGTH+MAX_FIELD_WIDTH], char min_key[MAX_KEY_LENGTH+MAX_FIELD_WIDTH],
max_key[MAX_KEY_LENGTH+MAX_FIELD_WIDTH]; max_key[MAX_KEY_LENGTH+MAX_FIELD_WIDTH];
bool quick; // Don't calulate possible keys bool quick; // Don't calulate possible keys
char escape; COND *cond;
} PARAM; } PARAM;
static SEL_TREE * get_mm_parts(PARAM *param,Field *field, static SEL_TREE * get_mm_parts(PARAM *param,Field *field,
...@@ -638,9 +638,6 @@ int SQL_SELECT::test_quick_select(THD *thd, key_map keys_to_use, ...@@ -638,9 +638,6 @@ int SQL_SELECT::test_quick_select(THD *thd, key_map keys_to_use,
param.table=head; param.table=head;
param.keys=0; param.keys=0;
param.mem_root= &alloc; param.mem_root= &alloc;
if (cond->type() == Item::FUNC_ITEM &&
((Item_func*)cond)->functype() == Item_func::LIKE_FUNC)
param.escape= ((Item_func_like*)cond)->escape;
param.thd->no_errors=1; // Don't warn about NULL param.thd->no_errors=1; // Don't warn about NULL
init_sql_alloc(&alloc, param.thd->variables.range_alloc_block_size, 0); init_sql_alloc(&alloc, param.thd->variables.range_alloc_block_size, 0);
if (!(param.key_parts = (KEY_PART*) alloc_root(&alloc, if (!(param.key_parts = (KEY_PART*) alloc_root(&alloc,
...@@ -821,6 +818,8 @@ static SEL_TREE *get_mm_tree(PARAM *param,COND *cond) ...@@ -821,6 +818,8 @@ static SEL_TREE *get_mm_tree(PARAM *param,COND *cond)
if (cond_func->select_optimize() == Item_func::OPTIMIZE_NONE) if (cond_func->select_optimize() == Item_func::OPTIMIZE_NONE)
DBUG_RETURN(0); // Can't be calculated DBUG_RETURN(0); // Can't be calculated
param->cond= cond;
if (cond_func->functype() == Item_func::BETWEEN) if (cond_func->functype() == Item_func::BETWEEN)
{ {
if (cond_func->arguments()[0]->type() == Item::FIELD_ITEM) if (cond_func->arguments()[0]->type() == Item::FIELD_ITEM)
...@@ -997,23 +996,26 @@ get_mm_leaf(PARAM *param, Field *field, KEY_PART *key_part, ...@@ -997,23 +996,26 @@ get_mm_leaf(PARAM *param, Field *field, KEY_PART *key_part,
if (maybe_null) if (maybe_null)
max_str[0]= min_str[0]=0; max_str[0]= min_str[0]=0;
if (field->binary()) if (field->binary())
like_error=like_range(res->ptr(),res->length(),param->escape,field_length, like_error=like_range(res->ptr(), res->length(),
min_str+offset,max_str+offset,(char) 255, ((Item_func_like*)(param->cond))->escape,
&min_length,&max_length); field_length, min_str + offset, max_str + offset,
(char) 255, &min_length, &max_length);
else else
{ {
#ifdef USE_STRCOLL #ifdef USE_STRCOLL
if (use_strcoll(default_charset_info)) if (use_strcoll(default_charset_info))
like_error= my_like_range(default_charset_info, like_error= my_like_range(default_charset_info, res->ptr(),
res->ptr(),res->length(),param->escape, res->length(),
field_length, min_str+maybe_null, ((Item_func_like*)(param->cond))->escape,
max_str+maybe_null,&min_length,&max_length); field_length, min_str + maybe_null,
max_str + maybe_null, &min_length,
&max_length);
else else
#endif #endif
like_error=like_range(res->ptr(),res->length(),param->escape, like_error=like_range(res->ptr(), res->length(),
field_length, ((Item_func_like*)(param->cond))->escape,
min_str+offset,max_str+offset, field_length, min_str + offset, max_str + offset,
max_sort_char,&min_length,&max_length); max_sort_char, &min_length, &max_length);
} }
if (like_error) // Can't optimize with LIKE if (like_error) // Can't optimize with LIKE
DBUG_RETURN(0); DBUG_RETURN(0);
......
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