← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to plugin/heap/hp_rnext.cc

  • Committer: Jay Pipes
  • Date: 2010-03-09 20:02:29 UTC
  • mto: This revision was merged to the branch mainline in revision 1339.
  • Revision ID: jpipes@serialcoder-20100309200229-dfrliy4fads9vyf4
https://launchpad.net/bugs/535296
Fixes Bug #535296 by only incrementing ha_commit_count when its a normal transaction commit.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin/heap/hp_rnext.cc
11
11
 
12
12
   You should have received a copy of the GNU General Public License
13
13
   along with this program; if not, write to the Free Software
14
 
   Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA */
 
14
   Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA */
15
15
 
16
16
#include "heap_priv.h"
17
17
 
24
24
int heap_rnext(HP_INFO *info, unsigned char *record)
25
25
{
26
26
  unsigned char *pos;
27
 
  HP_SHARE *share=info->getShare();
 
27
  HP_SHARE *share=info->s;
28
28
  HP_KEYDEF *keyinfo;
29
29
 
30
30
  if (info->lastinx < 0)
31
31
    return(errno=HA_ERR_WRONG_INDEX);
32
32
 
33
33
  keyinfo = share->keydef + info->lastinx;
 
34
  if (keyinfo->algorithm == HA_KEY_ALG_BTREE)
 
35
  {
 
36
    heap_rb_param custom_arg;
 
37
 
 
38
    if (info->last_pos)
 
39
    {
 
40
      /*
 
41
        We enter this branch for non-DELETE queries after heap_rkey()
 
42
        or heap_rfirst(). As last key position (info->last_pos) is available,
 
43
        we only need to climb the tree using tree_search_next().
 
44
      */
 
45
      pos = (unsigned char *)tree_search_next(&keyinfo->rb_tree,
 
46
                                              &info->last_pos,
 
47
                                              offsetof(TREE_ELEMENT, left),
 
48
                                              offsetof(TREE_ELEMENT, right));
 
49
    }
 
50
    else if (!info->lastkey_len)
 
51
    {
 
52
      /*
 
53
        We enter this branch only for DELETE queries after heap_rfirst(). E.g.
 
54
        DELETE FROM t1 WHERE a<10. As last key position is not available
 
55
        (last key is removed by heap_delete()), we must restart search as it
 
56
        is done in heap_rfirst().
 
57
 
 
58
        It should be safe to handle this situation without this branch. That is
 
59
        branch below should find smallest element in a tree as lastkey_len is
 
60
        zero. tree_search_edge() is a kind of optimisation here as it should be
 
61
        faster than tree_search_key().
 
62
      */
 
63
      pos= (unsigned char *)tree_search_edge(&keyinfo->rb_tree, info->parents,
 
64
                                             &info->last_pos,
 
65
                                             offsetof(TREE_ELEMENT, left));
 
66
    }
 
67
    else
 
68
    {
 
69
      /*
 
70
        We enter this branch only for DELETE queries after heap_rkey(). E.g.
 
71
        DELETE FROM t1 WHERE a=10. As last key position is not available
 
72
        (last key is removed by heap_delete()), we must restart search as it
 
73
        is done in heap_rkey().
 
74
      */
 
75
      custom_arg.keyseg = keyinfo->seg;
 
76
      custom_arg.key_length = info->lastkey_len;
 
77
      custom_arg.search_flag = SEARCH_SAME | SEARCH_FIND;
 
78
      pos = (unsigned char *)tree_search_key(&keyinfo->rb_tree,
 
79
                                             info->lastkey, info->parents,
 
80
                                             &info->last_pos,
 
81
                                             info->last_find_flag,
 
82
                                             &custom_arg);
 
83
    }
 
84
    if (pos)
 
85
    {
 
86
      memcpy(&pos, pos + (*keyinfo->get_key_length)(keyinfo, pos),
 
87
             sizeof(unsigned char*));
 
88
      info->current_ptr = pos;
 
89
    }
 
90
    else
 
91
    {
 
92
      errno = HA_ERR_KEY_NOT_FOUND;
 
93
    }
 
94
  }
 
95
  else
34
96
  {
35
97
    if (info->current_hash_ptr)
36
 
      pos= hp_search_next(info, keyinfo, &info->lastkey[0],
 
98
      pos= hp_search_next(info, keyinfo, info->lastkey,
37
99
                           info->current_hash_ptr);
38
100
    else
39
101
    {
43
105
        errno=HA_ERR_KEY_NOT_FOUND;
44
106
      }
45
107
      else if (!info->current_ptr)              /* Deleted or first call */
46
 
        pos= hp_search(info, keyinfo, &info->lastkey[0], 0);
 
108
        pos= hp_search(info, keyinfo, info->lastkey, 0);
47
109
      else
48
 
        pos= hp_search(info, keyinfo, &info->lastkey[0], 1);
 
110
        pos= hp_search(info, keyinfo, info->lastkey, 1);
49
111
    }
50
112
  }
51
113
  if (!pos)