← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/create_field.h

  • 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:
drizzled/create_field.h
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-2009 Sun Microsystems, Inc.
 
4
 *  Copyright (C) 2008-2009 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
20
20
#ifndef DRIZZLED_CREATE_FIELD_H
21
21
#define DRIZZLED_CREATE_FIELD_H
22
22
 
23
 
#include <drizzled/field.h>
 
23
#include "drizzled/field.h"
24
24
 
25
25
namespace drizzled
26
26
{
 
27
 
27
28
class Item;
28
29
typedef struct st_typelib TYPELIB;
29
30
 
42
43
  LEX_STRING comment; /**< A comment for this field */
43
44
  Item *def; /**< Default value for the new field */
44
45
  enum enum_field_types sql_type; /**< The data type of the new field */
45
 
 
46
 
  enum_field_types type() const
47
 
  {
48
 
    return sql_type;
49
 
  }
50
 
 
51
46
  /**
52
47
   * At various stages in execution this can be length of field in bytes or
53
48
   * max number of characters.
131
126
            enum column_format_type column_format);
132
127
};
133
128
 
134
 
std::ostream& operator<<(std::ostream& output, const CreateField &field);
135
 
 
136
129
} /* namespace drizzled */
137
130
 
138
131
#endif /* DRIZZLED_CREATE_FIELD_H */