← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to plugin/myisam/mi_create.cc

[patch 125/129] Merge patch for revision 1943 from InnoDB SVN:
revno: 1943
revision-id: svn-v4:16c675df-0fcb-4bc9-8058-dcc011a37293:branches/zip:6207
parent: svn-v4:16c675df-0fcb-4bc9-8058-dcc011a37293:branches/zip:6202
committer: vasil
timestamp: Fri 2009-11-20 08:19:14 +0000
message:
  branches/zip: Merge r6198:6206 from branches/5.1:
  
  (r6203 was skipped as it is already in branches/zip)
  
    ------------------------------------------------------------------------
    r6200 | vasil | 2009-11-19 12:14:23 +0200 (Thu, 19 Nov 2009) | 4 lines
    Changed paths:
       M /branches/5.1/btr/btr0btr.c
    
    branches/5.1:
    
    White space fixup - indent under the opening (
    
    ------------------------------------------------------------------------
    r6203 | jyang | 2009-11-19 15:12:22 +0200 (Thu, 19 Nov 2009) | 8 lines
    Changed paths:
       M /branches/5.1/btr/btr0btr.c
    
    branches/5.1: Use btr_free_root() instead of fseg_free() for
    the fix of bug #48469, because fseg_free() is not defined
    in the zip branch. And we could save one mini-trasaction started
    by fseg_free().
    
    Approved by Marko.
    
    
    ------------------------------------------------------------------------
    r6205 | jyang | 2009-11-20 07:55:48 +0200 (Fri, 20 Nov 2009) | 11 lines
    Changed paths:
       M /branches/5.1/handler/ha_innodb.cc
    
    branches/5.1: Add a special case to handle the Duplicated Key error
    and return DB_ERROR instead. This is to avoid a possible SIGSEGV
    by mysql error handling re-entering the storage layer for dup key
    info without proper table handle.
    This is to prevent a server crash when error situation in bug
    #45961 "DDL on partitioned innodb tables leaves data dictionary
    in an inconsistent state" happens.
    
    rb://157 approved by Sunny Bains.
    
    
    ------------------------------------------------------------------------
    r6206 | jyang | 2009-11-20 09:38:43 +0200 (Fri, 20 Nov 2009) | 5 lines
    Changed paths:
       M /branches/5.1/handler/ha_innodb.cc
    
    branches/5.1: Fix a minor code formating issue for 
    the parenthesis iplacement of the if condition in
    rename_table().
    
    
    ------------------------------------------------------------------------
modified:
  handler/ha_innodb.cc           2@16c675df-0fcb-4bc9-8058-dcc011a37293:trunk%2Fhandler%2Fha_innodb.cc
diff:
=== modified file 'handler/ha_innodb.cc'

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin/myisam/mi_create.cc
16
16
/* Create a MyISAM table */
17
17
 
18
18
#include "myisam_priv.h"
19
 
#include <drizzled/internal/my_bit.h>
20
 
#include <drizzled/internal/my_sys.h>
 
19
#include "drizzled/internal/my_bit.h"
 
20
#include "drizzled/internal/my_sys.h"
21
21
 
22
 
#include <drizzled/util/test.h>
23
 
#include <drizzled/global_charset_info.h>
24
 
#include <drizzled/error.h>
 
22
#include "drizzled/util/test.h"
 
23
#include "drizzled/global_charset_info.h"
 
24
#include "drizzled/error.h"
25
25
 
26
26
#include <cassert>
27
27
#include <algorithm>
392
392
  /* There are only 16 bits for the total header length. */
393
393
  if (info_length > 65535)
394
394
  {
395
 
    my_printf_error(EE_OK, "MyISAM table '%s' has too many columns and/or "
 
395
    my_printf_error(0, "MyISAM table '%s' has too many columns and/or "
396
396
                    "indexes and/or unique constraints.",
397
397
                    MYF(0), name + internal::dirname_length(name));
398
398
    errno= HA_WRONG_CREATE_OPTION;
526
526
  */
527
527
  if (test_if_reopen(filename))
528
528
  {
529
 
    my_printf_error(EE_OK, "MyISAM table '%s' is in use "
 
529
    my_printf_error(0, "MyISAM table '%s' is in use "
530
530
                    "(most likely by a MERGE table). Try FLUSH TABLES.",
531
531
                    MYF(0), name + internal::dirname_length(name));
532
532
    errno= HA_ERR_TABLE_EXIST;