← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to tests/t/synchronization.test

  • Committer: Padraig O'Sullivan
  • Date: 2009-06-29 17:24:02 UTC
  • mto: This revision was merged to the branch mainline in revision 1081.
  • Revision ID: osullivan.padraig@gmail.com-20090629172402-9c5n1kr7ry7xgau7
Removed the dependency on knowing the position of an I_S table in the
schema_tables array defined in show.cc. This issue crops up in
prepare_schema_table. An issue with my design is that it increases the time
complexity from O(1) to O(n) in numerous places to determine an I_S table to
work on since we don't have an explicit index into the array and instead
need to search by name. However, as n is the number of I_S tables and this
number is quite small (at the moment n is < 30), we don't see this causing
any issue. This design makes the code much more maintainable and easier to
understand. Previously, modifying anything to do with the I_S tables meant
having to tip-toe around the issue of hard-coded indexes into the
schema_tables array.

Show diffs side-by-side

added added

removed removed

Lines of Context:
tests/t/synchronization.test
21
21
  connection con2;
22
22
    CREATE TABLE t2 LIKE t1;
23
23
    replace_result x1 xx x2 xx;
 
24
    --replace_regex /ENGINE=[a-zA-Z]+/ENGINE=DEFAULT/
24
25
    SHOW CREATE TABLE t2;
25
26
    DROP TABLE t2;
26
27
  connection con1;
29
30
  connection con2;
30
31
    CREATE TABLE t2 LIKE t1;
31
32
    replace_result x1 xx x2 xx;
 
33
    --replace_regex /ENGINE=[a-zA-Z]+/ENGINE=DEFAULT/
32
34
    SHOW CREATE TABLE t2;
33
35
    DROP TABLE t2;
34
36
  connection con1;