Bug 981487 - change bugs_fulltext from myisam to innodb

parent 7f3a749d
...@@ -838,13 +838,10 @@ sub create { ...@@ -838,13 +838,10 @@ sub create {
Bugzilla::Hook::process('bug_end_of_create', Bugzilla::Hook::process('bug_end_of_create',
{bug => $bug, timestamp => $timestamp,}); {bug => $bug, timestamp => $timestamp,});
$dbh->bz_commit_transaction();
# Because MySQL doesn't support transactions on the fulltext table,
# we do this after we've committed the transaction. That way we're
# sure we're inserting a good Bug ID.
$bug->_sync_fulltext(new_bug => 1); $bug->_sync_fulltext(new_bug => 1);
$dbh->bz_commit_transaction();
return $bug; return $bug;
} }
...@@ -1211,17 +1208,13 @@ sub update { ...@@ -1211,17 +1208,13 @@ sub update {
delete $user->{bugs_ignored} if $bug_ignored_changed; delete $user->{bugs_ignored} if $bug_ignored_changed;
} }
$dbh->bz_commit_transaction();
# The only problem with this here is that update() is often called
# in the middle of a transaction, and if that transaction is rolled
# back, this change will *not* be rolled back. As we expect rollbacks
# to be extremely rare, that is OK for us.
$self->_sync_fulltext( $self->_sync_fulltext(
update_short_desc => $changes->{short_desc}, update_short_desc => $changes->{short_desc},
update_comments => $self->{added_comments} || $self->{comment_isprivate} update_comments => $self->{added_comments} || $self->{comment_isprivate}
); );
$dbh->bz_commit_transaction();
# Remove obsolete internal variables. # Remove obsolete internal variables.
delete $self->{'_old_assigned_to'}; delete $self->{'_old_assigned_to'};
delete $self->{'_old_qa_contact'}; delete $self->{'_old_qa_contact'};
......
...@@ -85,7 +85,7 @@ use constant REVERSE_MAPPING => { ...@@ -85,7 +85,7 @@ use constant REVERSE_MAPPING => {
# as in their db-specific version, so no reverse mapping is needed. # as in their db-specific version, so no reverse mapping is needed.
}; };
use constant MYISAM_TABLES => qw(bugs_fulltext); use constant MYISAM_TABLES => qw();
#------------------------------------------------------------------------------ #------------------------------------------------------------------------------
sub _initialize { sub _initialize {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment