← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/sql_parse.cc

  • 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:
drizzled/sql_parse.cc
317
317
    It prepares a Select_Lex and a TableList object to represent the
318
318
    given command as a SELECT parse tree.
319
319
 
320
 
  @param session              thread handle
321
 
  @param lex              current lex
322
 
  @param table_ident      table alias if it's used
323
 
  @param schema_table_idx the type of the INFORMATION_SCHEMA table to be
324
 
                          created
 
320
  @param session           thread handle
 
321
  @param lex               current lex
 
322
  @param table_ident       table alias if it's used
 
323
  @param schema_table_name the name of the INFORMATION_SCHEMA table to be
 
324
                           created
325
325
 
326
326
  @note
327
327
    Due to the way this function works with memory and LEX it cannot
336
336
*/
337
337
 
338
338
int prepare_schema_table(Session *session, LEX *lex, Table_ident *table_ident,
339
 
                         enum enum_schema_tables schema_table_idx)
 
339
                         const string& schema_table_name)
340
340
{
341
341
  Select_Lex *schema_select_lex= NULL;
342
342
 
343
 
  switch (schema_table_idx) {
344
 
  case SCH_SCHEMATA:
345
 
    break;
346
 
  case SCH_TABLE_NAMES:
347
 
  case SCH_TABLES:
348
 
    {
349
 
      LEX_STRING db;
350
 
      size_t dummy;
351
 
      if (lex->select_lex.db == NULL &&
352
 
          lex->copy_db_to(&lex->select_lex.db, &dummy))
353
 
      {
354
 
        return(1);
355
 
      }
356
 
      schema_select_lex= new Select_Lex();
357
 
      db.str= schema_select_lex->db= lex->select_lex.db;
358
 
      schema_select_lex->table_list.first= NULL;
359
 
      db.length= strlen(db.str);
360
 
 
361
 
      if (check_db_name(&db))
362
 
      {
363
 
        my_error(ER_WRONG_DB_NAME, MYF(0), db.str);
364
 
        return(1);
365
 
      }
366
 
      break;
367
 
    }
368
 
  case SCH_COLUMNS:
369
 
  case SCH_STATISTICS:
 
343
 
 
344
  if (schema_table_name.compare("TABLES") == 0 ||
 
345
      schema_table_name.compare("TABLE_NAMES") == 0)
 
346
  {
 
347
    LEX_STRING db;
 
348
    size_t dummy;
 
349
    if (lex->select_lex.db == NULL &&
 
350
        lex->copy_db_to(&lex->select_lex.db, &dummy))
 
351
    {
 
352
      return (1);
 
353
    }
 
354
    schema_select_lex= new Select_Lex();
 
355
    db.str= schema_select_lex->db= lex->select_lex.db;
 
356
    schema_select_lex->table_list.first= NULL;
 
357
    db.length= strlen(db.str);
 
358
 
 
359
    if (check_db_name(&db))
 
360
    {
 
361
      my_error(ER_WRONG_DB_NAME, MYF(0), db.str);
 
362
      return (1);
 
363
    }
 
364
  }
 
365
  else if (schema_table_name.compare("COLUMNS") == 0 ||
 
366
           schema_table_name.compare("STATISTICS") == 0)
370
367
  {
371
368
    assert(table_ident);
372
369
    TableList **query_tables_last= lex->query_tables_last;
374
371
    /* 'parent_lex' is used in init_query() so it must be before it. */
375
372
    schema_select_lex->parent_lex= lex;
376
373
    schema_select_lex->init_query();
377
 
    if (!schema_select_lex->add_table_to_list(session, table_ident, 0, 0, TL_READ))
378
 
      return(1);
 
374
    if (! schema_select_lex->add_table_to_list(session, table_ident, 0, 0, TL_READ))
 
375
    {
 
376
      return (1);
 
377
    }
379
378
    lex->query_tables_last= query_tables_last;
380
 
    break;
381
 
  }
382
 
  case SCH_OPEN_TABLES:
383
 
  case SCH_VARIABLES:
384
 
  case SCH_STATUS:
385
 
  default:
386
 
    break;
387
379
  }
388
380
 
389
381
  Select_Lex *select_lex= lex->current_select;
390
382
  assert(select_lex);
391
 
  if (make_schema_select(session, select_lex, schema_table_idx))
 
383
  if (make_schema_select(session, select_lex, schema_table_name))
392
384
  {
393
385
    return(1);
394
386
  }