Commit d098596b authored by Sergey Petrunya's avatar Sergey Petrunya

Fix a number of problems with reverse buffer use

parent 8d07c16a
...@@ -644,6 +644,8 @@ int DsMrr_impl::dsmrr_fill_rowid_buffer() ...@@ -644,6 +644,8 @@ int DsMrr_impl::dsmrr_fill_rowid_buffer()
rowid_buffer.reset_for_writing(); rowid_buffer.reset_for_writing();
identical_rowid_ptr= NULL; identical_rowid_ptr= NULL;
if (do_sort_keys)
key_buffer.flip();
while (rowid_buffer.have_space_for(rowid_buff_elem_size)) while (rowid_buffer.have_space_for(rowid_buff_elem_size))
{ {
...@@ -860,13 +862,16 @@ void DsMrr_impl::dsmrr_fill_key_buffer() ...@@ -860,13 +862,16 @@ void DsMrr_impl::dsmrr_fill_key_buffer()
setup_buffer_sizes(&cur_range.start_key); setup_buffer_sizes(&cur_range.start_key);
} }
if (key_buffer.is_reverse() && is_mrr_assoc)
key_buffer.write((uchar*)&cur_range.ptr, sizeof(void*));
/* Put key, or {key, range_id} pair into the buffer */ /* Put key, or {key, range_id} pair into the buffer */
if (use_key_pointers) if (use_key_pointers)
key_buffer.write((uchar*)&cur_range.start_key.key, sizeof(char*)); key_buffer.write((uchar*)&cur_range.start_key.key, sizeof(char*));
else else
key_buffer.write(cur_range.start_key.key, key_tuple_length); key_buffer.write(cur_range.start_key.key, key_tuple_length);
if (is_mrr_assoc) if (!key_buffer.is_reverse() && is_mrr_assoc)
key_buffer.write((uchar*)&cur_range.ptr, sizeof(void*)); key_buffer.write((uchar*)&cur_range.ptr, sizeof(void*));
} }
...@@ -966,7 +971,6 @@ int DsMrr_impl::dsmrr_next_from_index(char **range_info_arg) ...@@ -966,7 +971,6 @@ int DsMrr_impl::dsmrr_next_from_index(char **range_info_arg)
} }
goto check_record; goto check_record;
// goto read_and_check;
} }
while(1) while(1)
...@@ -999,6 +1003,7 @@ int DsMrr_impl::dsmrr_next_from_index(char **range_info_arg) ...@@ -999,6 +1003,7 @@ int DsMrr_impl::dsmrr_next_from_index(char **range_info_arg)
res= HA_ERR_END_OF_FILE; res= HA_ERR_END_OF_FILE;
goto end; goto end;
} }
if (!do_rowid_fetch)
dsmrr_fill_key_buffer(); dsmrr_fill_key_buffer();
if (!key_buffer.have_data(key_buff_elem_size)) if (!key_buffer.have_data(key_buff_elem_size))
{ {
......
...@@ -102,14 +102,25 @@ class SimpleBuffer ...@@ -102,14 +102,25 @@ class SimpleBuffer
{ {
*unused_start= start; *unused_start= start;
*unused_end= read_pos; *unused_end= read_pos;
start= read_pos;
} }
else else
{ {
*unused_start=read_pos; *unused_start= read_pos;
*unused_end=end; *unused_end= end;
end= read_pos;
} }
} }
void flip()
{
uchar *tmp= read_pos;
read_pos= write_pos;
write_pos= tmp;
direction= -direction;
}
bool is_reverse() { return direction == -1; }
void grow(uchar *unused_start, uchar *unused_end) void grow(uchar *unused_start, uchar *unused_end)
{ {
/* /*
......
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