mirror of
https://github.com/MariaDB/server.git
synced 2025-01-17 04:22:27 +01:00
Bug fix: we should set have_rtree value in case of table reopen.
Code cleanup. myisam/mi_open.c: Bug fix: we should set have_rtree value in case of table reopen. myisam/myisamdef.h: code cleanup myisam/rt_index.c: code cleanup
This commit is contained in:
parent
19f00e0abe
commit
75d1d877b3
3 changed files with 11 additions and 7 deletions
|
@ -484,6 +484,7 @@ MI_INFO *mi_open(const char *name, int mode, uint open_flags)
|
|||
if (mi_open_datafile(&info, share, old_info->dfile))
|
||||
goto err;
|
||||
errpos=5;
|
||||
have_rtree= old_info->rtree_recursion_state != NULL;
|
||||
}
|
||||
|
||||
/* alloc and set up private structure parts */
|
||||
|
@ -492,13 +493,16 @@ MI_INFO *mi_open(const char *name, int mode, uint open_flags)
|
|||
&info.blobs,sizeof(MI_BLOB)*share->base.blobs,
|
||||
&info.buff,(share->base.max_key_block_length*2+
|
||||
share->base.max_key_length),
|
||||
&info.rtree_recursion_state,have_rtree ? 1024 : 0,
|
||||
&info.lastkey,share->base.max_key_length*3+1,
|
||||
&info.filename,strlen(org_name)+1,
|
||||
&info.rtree_recursion_state,have_rtree ? 1024 : 0,
|
||||
NullS))
|
||||
goto err;
|
||||
errpos=6;
|
||||
|
||||
if (!have_rtree)
|
||||
info.rtree_recursion_state= NULL;
|
||||
|
||||
strmov(info.filename,org_name);
|
||||
memcpy(info.blobs,share->blobs,sizeof(MI_BLOB)*share->base.blobs);
|
||||
info.lastkey2=info.lastkey+share->base.max_key_length;
|
||||
|
|
|
@ -53,7 +53,7 @@ static int rtree_find_req(MI_INFO *info, MI_KEYDEF *keyinfo, uint search_flag, u
|
|||
int res;
|
||||
uchar *page_buf;
|
||||
int k_len;
|
||||
int *saved_key = (int*)(info->rtree_recursion_state + level * sizeof(int));
|
||||
uint *saved_key = (uint*) (info->rtree_recursion_state) + level;
|
||||
|
||||
if (!(page_buf = (uchar*)my_alloca((uint)keyinfo->block_length)))
|
||||
{
|
||||
|
@ -114,7 +114,7 @@ static int rtree_find_req(MI_INFO *info, MI_KEYDEF *keyinfo, uint search_flag, u
|
|||
uchar *after_key = rt_PAGE_NEXT_KEY(k, k_len, nod_flag);
|
||||
info->lastpos = _mi_dpos(info, 0, after_key);
|
||||
info->lastkey_length = k_len + info->s->base.rec_reflength;
|
||||
memcpy(info->lastkey, k, k_len + info->s->base.rec_reflength);
|
||||
memcpy(info->lastkey, k, info->lastkey_length);
|
||||
|
||||
info->rtree_recursion_depth = level;
|
||||
*saved_key = k - page_buf;
|
||||
|
@ -205,7 +205,7 @@ int rtree_find_next(MI_INFO *info, uint keynr, uint search_flag)
|
|||
|
||||
info->lastpos = _mi_dpos(info, 0, after_key);
|
||||
info->lastkey_length = k_len + info->s->base.rec_reflength;
|
||||
memcpy(info->lastkey, key, k_len + info->s->base.rec_reflength);
|
||||
memcpy(info->lastkey, key, info->lastkey_length);
|
||||
|
||||
*(int*)info->int_keypos = key - info->buff;
|
||||
if (after_key >= info->int_maxpos)
|
||||
|
@ -246,7 +246,7 @@ static int rtree_get_req(MI_INFO *info, MI_KEYDEF *keyinfo, uint key_length,
|
|||
int res;
|
||||
uchar *page_buf;
|
||||
uint k_len;
|
||||
int *saved_key = (int*)(info->rtree_recursion_state + level*sizeof(int));
|
||||
uint *saved_key = (uint*) (info->rtree_recursion_state) + level;
|
||||
|
||||
if (!(page_buf = (uchar*)my_alloca((uint)keyinfo->block_length)))
|
||||
return -1;
|
||||
|
@ -297,7 +297,7 @@ static int rtree_get_req(MI_INFO *info, MI_KEYDEF *keyinfo, uint key_length,
|
|||
uchar *after_key = rt_PAGE_NEXT_KEY(k, k_len, nod_flag);
|
||||
info->lastpos = _mi_dpos(info, 0, after_key);
|
||||
info->lastkey_length = k_len + info->s->base.rec_reflength;
|
||||
memcpy(info->lastkey, k, k_len + info->s->base.rec_reflength);
|
||||
memcpy(info->lastkey, k, info->lastkey_length);
|
||||
|
||||
info->rtree_recursion_depth = level;
|
||||
*saved_key = k - page_buf;
|
||||
|
|
Loading…
Reference in a new issue