← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to tests/include/diff_tables.inc

  • 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:
tests/include/diff_tables.inc
56
56
disable_query_log;
57
57
 
58
58
--error 0,1
59
 
--remove_file $DRIZZLETEST_VARDIR/tmp/diff_table_1
 
59
--remove_file $MYSQLTEST_VARDIR/tmp/diff_table_1
60
60
--error 0,1
61
 
--remove_file $DRIZZLETEST_VARDIR/tmp/diff_table_2
 
61
--remove_file $MYSQLTEST_VARDIR/tmp/diff_table_2
62
62
 
63
63
let $_diff_table=$diff_table_2;
64
64
let $_diff_i=2;
102
102
  # Now that we have the comma-separated list of columns, we can write
103
103
  # the table to a file.
104
104
  eval SELECT * FROM $_diff_table ORDER BY 1 
105
 
              INTO OUTFILE '$DRIZZLETEST_VARDIR/tmp/diff_table_$_diff_i';
 
105
              INTO OUTFILE '$MYSQLTEST_VARDIR/tmp/diff_table_$_diff_i';
106
106
 
107
107
  # Do the same for $diff_table_1.
108
108
  dec $_diff_i;
112
112
 
113
113
# ==== Compare the generated files ====
114
114
 
115
 
diff_files $DRIZZLETEST_VARDIR/tmp/diff_table_1 $DRIZZLETEST_VARDIR/tmp/diff_table_2;
 
115
diff_files $MYSQLTEST_VARDIR/tmp/diff_table_1 $MYSQLTEST_VARDIR/tmp/diff_table_2;
116
116
 
117
 
--remove_file $DRIZZLETEST_VARDIR/tmp/diff_table_1
118
 
--remove_file $DRIZZLETEST_VARDIR/tmp/diff_table_2
 
117
--remove_file $MYSQLTEST_VARDIR/tmp/diff_table_1
 
118
--remove_file $MYSQLTEST_VARDIR/tmp/diff_table_2
119
119
 
120
120
enable_query_log;