From d247ef1bcd7506152b99e091f4a8f4251b78c627 Mon Sep 17 00:00:00 2001 From: Byron Jones Date: Mon, 17 Aug 2015 22:29:08 +0800 Subject: Bug 981487 - change bugs_fulltext from myisam to innodb --- Bugzilla/Bug.pm | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) (limited to 'Bugzilla/Bug.pm') diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm index 389747011..9e4135d1c 100644 --- a/Bugzilla/Bug.pm +++ b/Bugzilla/Bug.pm @@ -832,13 +832,11 @@ sub create { stash => $stash, }); - $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 ); + $dbh->bz_commit_transaction(); + # BMO - some work should happen outside of the transaction block Bugzilla::Hook::process('bug_after_create', { bug => $bug, timestamp => $timestamp }); @@ -1178,17 +1176,13 @@ sub update { 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( update_short_desc => $changes->{short_desc}, update_comments => $self->{added_comments} || $self->{comment_isprivate} ); + $dbh->bz_commit_transaction(); + # Remove obsolete internal variables. delete $self->{'_old_assigned_to'}; delete $self->{'_old_qa_contact'}; -- cgit v1.2.3-24-g4f1b