← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to plugin/myisam/mi_create.cc

  • Committer: Jay Pipes
  • Date: 2010-03-09 20:02:29 UTC
  • mto: This revision was merged to the branch mainline in revision 1339.
  • Revision ID: jpipes@serialcoder-20100309200229-dfrliy4fads9vyf4
https://launchpad.net/bugs/535296
Fixes Bug #535296 by only incrementing ha_commit_count when its a normal transaction commit.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin/myisam/mi_create.cc
11
11
 
12
12
   You should have received a copy of the GNU General Public License
13
13
   along with this program; if not, write to the Free Software
14
 
   Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA */
 
14
   Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA */
15
15
 
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>
265
265
        case HA_KEYTYPE_ULONG_INT:
266
266
        case HA_KEYTYPE_LONGLONG:
267
267
        case HA_KEYTYPE_ULONGLONG:
 
268
        case HA_KEYTYPE_UINT24:
268
269
          keyseg->flag|= HA_SWAP_KEY;
269
270
          break;
270
271
        case HA_KEYTYPE_VARTEXT1:
392
393
  /* There are only 16 bits for the total header length. */
393
394
  if (info_length > 65535)
394
395
  {
395
 
    my_printf_error(EE_OK, "MyISAM table '%s' has too many columns and/or "
 
396
    my_printf_error(0, "MyISAM table '%s' has too many columns and/or "
396
397
                    "indexes and/or unique constraints.",
397
398
                    MYF(0), name + internal::dirname_length(name));
398
399
    errno= HA_WRONG_CREATE_OPTION;
467
468
  if (! (flags & HA_DONT_TOUCH_DATA))
468
469
    share.state.create_time= (long) time((time_t*) 0);
469
470
 
470
 
  THR_LOCK_myisam.lock();
 
471
  pthread_mutex_lock(&THR_LOCK_myisam);
471
472
 
472
473
  /*
473
474
    NOTE: For test_if_reopen() we need a real path name. Hence we need
526
527
  */
527
528
  if (test_if_reopen(filename))
528
529
  {
529
 
    my_printf_error(EE_OK, "MyISAM table '%s' is in use "
 
530
    my_printf_error(0, "MyISAM table '%s' is in use "
530
531
                    "(most likely by a MERGE table). Try FLUSH TABLES.",
531
532
                    MYF(0), name + internal::dirname_length(name));
532
533
    errno= HA_ERR_TABLE_EXIST;
670
671
      goto err;
671
672
  }
672
673
  errpos=0;
673
 
  THR_LOCK_myisam.unlock();
 
674
  pthread_mutex_unlock(&THR_LOCK_myisam);
674
675
  if (internal::my_close(file,MYF(0)))
675
676
    goto err;
676
677
  free((char*) rec_per_key_part);
677
678
  return(0);
678
679
 
679
680
err:
680
 
  THR_LOCK_myisam.unlock();
 
681
  pthread_mutex_unlock(&THR_LOCK_myisam);
681
682
  save_errno=errno;
682
683
  switch (errpos) {
683
684
  case 3: