diff options
author | mkanat%bugzilla.org <> | 2009-08-18 19:42:56 +0000 |
---|---|---|
committer | mkanat%bugzilla.org <> | 2009-08-18 19:42:56 +0000 |
commit | 56c9e8034aa591a50a7aa11dee2f7289e807c6f4 (patch) | |
tree | 0dd1005d64d621bee344d68599fd37dd4b0d44fa /Bugzilla | |
parent | dbfeceeba9f66601ff3145713cbb4050016b4411 (diff) | |
download | bugs-56c9e8034aa591a50a7aa11dee2f7289e807c6f4.tar bugs-56c9e8034aa591a50a7aa11dee2f7289e807c6f4.tar.gz bugs-56c9e8034aa591a50a7aa11dee2f7289e807c6f4.tar.bz2 bugs-56c9e8034aa591a50a7aa11dee2f7289e807c6f4.tar.xz bugs-56c9e8034aa591a50a7aa11dee2f7289e807c6f4.zip |
Bug 507969: Speed up checksetup for cases when it's doing nothing
Patch by Max Kanat-Alexander <mkanat@bugzilla.org> r=LpSolit, a=mkanat
Diffstat (limited to 'Bugzilla')
-rw-r--r-- | Bugzilla/DB/Mysql.pm | 25 | ||||
-rw-r--r-- | Bugzilla/Install/DB.pm | 19 |
2 files changed, 22 insertions, 22 deletions
diff --git a/Bugzilla/DB/Mysql.pm b/Bugzilla/DB/Mysql.pm index 380448ba3..39064e4e2 100644 --- a/Bugzilla/DB/Mysql.pm +++ b/Bugzilla/DB/Mysql.pm @@ -317,15 +317,11 @@ EOT } - # Figure out if any existing tables are of type ISAM and convert them - # to type MyISAM if so. ISAM tables are deprecated in MySQL 3.23, - # which Bugzilla now requires, and they don't support more than 16 - # indexes per table, which Bugzilla needs. - my $table_status = $self->selectall_arrayref("SHOW TABLE STATUS"); + my %table_status = @{ $self->selectcol_arrayref("SHOW TABLE STATUS", + {Columns=>[1,2]}) }; my @isam_tables; - foreach my $row (@$table_status) { - my ($name, $type) = @$row; - push(@isam_tables, $name) if $type eq "ISAM"; + foreach my $name (keys %table_status) { + push(@isam_tables, $name) if $table_status{$name} eq "ISAM"; } if(scalar(@isam_tables)) { @@ -347,7 +343,9 @@ EOT # We want to convert tables to InnoDB, but it's possible that they have # fulltext indexes on them, and conversion will fail unless we remove # the indexes. - if (grep($_ eq 'bugs', @tables)) { + if (grep($_ eq 'bugs', @tables) + and !grep($_ eq 'bugs_fulltext', @tables)) + { if ($self->bz_index_info_real('bugs', 'short_desc')) { $self->bz_drop_index_raw('bugs', 'short_desc'); } @@ -356,7 +354,9 @@ EOT $sd_index_deleted = 1; # Used for later schema cleanup. } } - if (grep($_ eq 'longdescs', @tables)) { + if (grep($_ eq 'longdescs', @tables) + and !grep($_ eq 'bugs_fulltext', @tables)) + { if ($self->bz_index_info_real('longdescs', 'thetext')) { $self->bz_drop_index_raw('longdescs', 'thetext'); } @@ -368,9 +368,8 @@ EOT # Upgrade tables from MyISAM to InnoDB my @myisam_tables; - foreach my $row (@$table_status) { - my ($name, $type) = @$row; - if ($type =~ /^MYISAM$/i + foreach my $name (keys %table_status) { + if ($table_status{$name} =~ /^MYISAM$/i && !grep($_ eq $name, Bugzilla::DB::Schema::Mysql::MYISAM_TABLES)) { push(@myisam_tables, $name) ; diff --git a/Bugzilla/Install/DB.pm b/Bugzilla/Install/DB.pm index 697e2fdcc..dd4b2fe1d 100644 --- a/Bugzilla/Install/DB.pm +++ b/Bugzilla/Install/DB.pm @@ -2340,10 +2340,11 @@ sub _initialize_dependency_tree_changes_email_pref { foreach my $desc (keys %events) { my $event = $events{$desc}; - my $sth = $dbh->prepare("SELECT COUNT(*) FROM email_setting - WHERE event = $event"); - $sth->execute(); - if (!($sth->fetchrow_arrayref()->[0])) { + my $have_events = $dbh->selectrow_array( + "SELECT 1 FROM email_setting WHERE event = $event " + . $dbh->sql_limit(1)); + + if (!$have_events) { # No settings in the table yet, so we assume that this is the # first time it's being set. print "Initializing \"$desc\" email_setting ...\n"; @@ -2901,11 +2902,8 @@ sub _initialize_workflow { # and mark these statuses as 'closed', even if some of these statuses are # expected to be open statuses. Bug statuses we have no information about # are left as 'open'. - my @closed_statuses = - @{$dbh->selectcol_arrayref('SELECT DISTINCT bug_status FROM bugs - WHERE resolution != ?', undef, '')}; - - # Append the default list of closed statuses *unless* we detect at least + # + # We append the default list of closed statuses *unless* we detect at least # one closed state in the DB (i.e. with is_open = 0). This would mean that # the DB has already been updated at least once and maybe the admin decided # that e.g. 'RESOLVED' is now an open state, in which case we don't want to @@ -2916,6 +2914,9 @@ sub _initialize_workflow { WHERE is_open = 0'); if (!$num_closed_states) { + my @closed_statuses = + @{$dbh->selectcol_arrayref('SELECT DISTINCT bug_status FROM bugs + WHERE resolution != ?', undef, '')}; @closed_statuses = map {$dbh->quote($_)} (@closed_statuses, qw(RESOLVED VERIFIED CLOSED)); |