← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/statement/create_index.cc

  • Committer: Prafulla Tekawade
  • Date: 2010-07-18 03:36:32 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-20100718033632-p7q6qtgliqbhe38p
Fix for Bug 592444

There were two problems:
o. In greedy_search optimizer method, best_extension_by_limited search
   maintains join embedding(nestedness) of tables added so far, so that 
   correct(valid)  join order is selected
   These are requirements from nested outer join executioner.
   The problem was, embedding_map was not correctly updated when a table 
   is added to optimal plan outside best_extension_by_limited search, 
   by greedy_search method. We need to update join->cur_embedding_map
   correctly here so that execution plan for other tables get
   generated.
   Invoked checked_interleaving_with_nj from greedy_search on the
   best_table selected. Fixed its prototype to take only one JoinTab
   This is same as mysql 5.1 source tree.
o. The other problem was, join->cur_embedding_map was not restored correctly
   when a table is added to the optimal plan to reflect the current embedding 
   map. 
   Taken good documented method restore_prev_nj_state which restores 
   cur_embedding_map from mysql 5.1 source tree and modified it for drizzled 
   code.

Show diffs side-by-side

added added

removed removed

Lines of Context:
drizzled/statement/create_index.cc
34
34
  TableList *all_tables= session->lex->query_tables;
35
35
 
36
36
  /* Chicken/Egg... we need to search for the table, to know if the table exists, so we can build a full identifier from it */
37
 
  message::table::shared_ptr original_table_message;
 
37
  message::Table original_table_message;
38
38
  {
39
 
    TableIdentifier identifier(first_table->getSchemaName(), first_table->getTableName());
 
39
    TableIdentifier identifier(first_table->db, first_table->table_name);
40
40
    if (plugin::StorageEngine::getTableDefinition(*session, identifier, original_table_message) != EEXIST)
41
41
    {
42
 
      std::string path;
43
 
      identifier.getSQLPath(path);
44
 
      my_error(ER_BAD_TABLE_ERROR, MYF(0), path.c_str());
 
42
      my_error(ER_BAD_TABLE_ERROR, MYF(0), identifier.getSQLPath().c_str());
45
43
      return true;
46
44
    }
47
45
  }
61
59
    return true;
62
60
  }
63
61
 
 
62
  create_info.row_type= ROW_TYPE_NOT_USED;
 
63
 
64
64
  bool res;
65
 
  if (original_table_message->type() == message::Table::STANDARD )
 
65
  if (original_table_message.type() == message::Table::STANDARD )
66
66
  {
67
 
    TableIdentifier identifier(first_table->getSchemaName(), first_table->getTableName());
 
67
    TableIdentifier identifier(first_table->db, first_table->table_name);
68
68
    create_info.default_table_charset= plugin::StorageEngine::getSchemaCollation(identifier);
69
69
 
70
70
    res= alter_table(session, 
71
71
                     identifier,
72
72
                     identifier,
73
73
                     &create_info, 
74
 
                     *original_table_message,
 
74
                     original_table_message,
75
75
                     create_table_message, 
76
76
                     first_table,
77
77
                     &alter_info,
78
 
                     0, (Order*) 0, 0);
 
78
                     0, (order_st*) 0, 0);
79
79
  }
80
80
  else
81
81
  {
82
 
    TableIdentifier catch22(first_table->getSchemaName(), first_table->getTableName());
83
 
    Table *table= session->find_temporary_table(catch22);
 
82
    Table *table= session->find_temporary_table(first_table);
84
83
    assert(table);
85
84
    {
86
 
      TableIdentifier identifier(first_table->getSchemaName(), first_table->getTableName(), table->getMutableShare()->getPath());
 
85
      TableIdentifier identifier(first_table->db, first_table->table_name, table->getMutableShare()->getPath());
87
86
      create_info.default_table_charset= plugin::StorageEngine::getSchemaCollation(identifier);
88
87
 
89
88
      res= alter_table(session, 
90
89
                       identifier,
91
90
                       identifier,
92
91
                       &create_info, 
93
 
                       *original_table_message,
 
92
                       original_table_message,
94
93
                       create_table_message, 
95
94
                       first_table,
96
95
                       &alter_info,
97
 
                       0, (Order*) 0, 0);
 
96
                       0, (order_st*) 0, 0);
98
97
    }
99
98
  }
100
99