Bug 299156: Reduce saniitycheck submitter_id time for large sites

Patch by Joel Peshkin <bugreport@peshkin.net>
r=mkanat, a=justdave

diff --git a/Bugzilla/DB/Schema.pm b/Bugzilla/DB/Schema.pm
index 52d1b3212d3b4c6153e7f3ea4d674d54db9807e7..300a97b98c130476f7152c94b78b3eaef14b6a4d 100644 (file)
--- a/Bugzilla/DB/Schema.pm
+++ b/Bugzilla/DB/Schema.pm
@@ -310,7 +310,7 @@ use constant ABSTRACT_SCHEMA => {
         INDEXES => [
             attachments_bug_id_idx => ['bug_id'],
             attachments_creation_ts_idx => ['creation_ts'],
-            attachments_submitter_id_idx => ['submitter_id'],
+            attachments_submitter_id_idx => ['submitter_id', 'bug_id'],
         ],
     },
 

diff --git a/checksetup.pl b/checksetup.pl
index b5158ccb3a862aa23f102d0148eccef306f4975f..ee2cd1e6705874d03e93a333d20bdcc308289864 100755 (executable)
--- a/checksetup.pl
+++ b/checksetup.pl
@@ -3770,8 +3770,6 @@ if ($emptygroupid) {
 }
 
 # 2005-02-12 bugreport@peshkin.net, bug 281787
-$dbh->bz_add_index('attachments', 'attachments_submitter_id_idx',
-                   [qw(submitter_id)]);
 $dbh->bz_add_index('bugs_activity', 'bugs_activity_who_idx', [qw(who)]);
 
 # This lastdiffed change and these default changes are unrelated,
@@ -3992,6 +3990,16 @@ AddGroup('editcomponents', 'Can create, destroy, and edit components.');
 AddGroup('editkeywords', 'Can create, destroy, and edit keywords.');
 AddGroup('admin', 'Administrators');
 
+# 2005-06-29 bugreport@peshkin.net, bug 299156
+if ($dbh->bz_index_info('attachments', 'attachments_submitter_id_idx') 
+   && (scalar(@{$dbh->bz_index_info('attachments', 
+                                    'attachments_submitter_id_idx'
+                                   )->{FIELDS}}) < 2)
+      ) {
+    $dbh->bz_drop_index('attachments', 'attachments_submitter_id_idx');
+}
+$dbh->bz_add_index('attachments', 'attachments_submitter_id_idx',
+                   [qw(submitter_id bug_id)]);
 
 if (!GroupDoesExist("editbugs")) {
     my $id = AddGroup('editbugs', 'Can edit all bug fields.', ".*");