~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to plugin/myisam/my_handler.cc

  • Committer: Prafulla Tekawade
  • Date: 2010-07-13 16:07:35 UTC
  • mto: (1662.1.4 rollup)
  • mto: This revision was merged to the branch mainline in revision 1664.
  • Revision ID: prafulla_t@users.sourceforge.net-20100713160735-2fsdtrm3azayuyu1
This bug is simillar to mysql bug 36133
http://bugs.mysql.com/bug.php?id=36133

Taking changes from that fix.

  - The problem was that the range optimizer evaluated constant expressions, 
    and among them it would try to evaluate IN-subquery predicates slated for
    handling with materialization strategy. However, these predicates require
    that parent_join->setup_subquery_materialization() is invoked before one
    attempts to evaluate them.
  
  - Fixed by making the range optimizer not to evaluate expressions that have
    item->is_expensive() == TRUE (these are materialization subqueries and 
    stored function calls). This should also resolve the problem that EXPLAIN 
    may be too long. 
    This change cuts off some opportunities for range optimizer, but this is 
    the price we're willing to pay for separation of query optimization and
    execution. 

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
1
/* -*- mode: c++; c-basic-offset: 2; indent-tabs-mode: nil; -*-
2
2
 *  vim:expandtab:shiftwidth=2:tabstop=2:smarttab:
3
3
 *
4
 
 *  Copyright (C) 2008 Sun Microsystems, Inc.
 
4
 *  Copyright (C) 2008 Sun Microsystems
5
5
 *
6
6
 *  This program is free software; you can redistribute it and/or modify
7
7
 *  it under the terms of the GNU General Public License as published by
17
17
 *  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
18
18
 */
19
19
 
20
 
#include <config.h>
 
20
#include "config.h"
21
21
 
22
 
#include <drizzled/charset_info.h>
 
22
#include "drizzled/charset_info.h"
23
23
#include <drizzled/base.h>
24
24
#include <plugin/myisam/my_handler.h>
25
 
#include <drizzled/internal/my_sys.h>
 
25
#include "drizzled/internal/my_sys.h"
26
26
 
27
27
#include <cassert>
28
28
#include <algorithm>
297
297
      a=  end;
298
298
      b+= 4; /* sizeof(long int); */
299
299
      break;
 
300
    case HA_KEYTYPE_UINT24:
 
301
      l_1=mi_uint3korr(a);
 
302
      l_2=mi_uint3korr(b);
 
303
      if (piks && (flag = CMP_NUM(l_1,l_2)))
 
304
        return ((keyseg->flag & HA_REVERSE_SORT) ? -flag : flag);
 
305
      a=  end;
 
306
      b+= 3;
 
307
      break;
300
308
    case HA_KEYTYPE_DOUBLE:
301
309
      mi_float8get(d_1,a);
302
310
      mi_float8get(d_2,b);
420
428
      }
421
429
    case HA_KEYTYPE_LONG_INT:
422
430
    case HA_KEYTYPE_ULONG_INT:
 
431
    case HA_KEYTYPE_UINT24:
423
432
    case HA_KEYTYPE_LONGLONG:
424
433
    case HA_KEYTYPE_ULONGLONG:
425
434
    case HA_KEYTYPE_DOUBLE: