revert Bug 981756: add support for "bigint unsigned" to db/schema and update bz_schema to reflect the current state

5 files changed, 7 insertions, 32 deletions

diff --git a/Bugzilla/DB/Schema.pm b/Bugzilla/DB/Schema.pm
index e2a7e138e..d8f3e175a 100644
--- a/Bugzilla/DB/Schema.pm
+++ b/Bugzilla/DB/Schema.pm
@@ -342,13 +342,13 @@ use constant ABSTRACT_SCHEMA => {
 
     bugs_activity => {
         FIELDS => [
-            id        => {TYPE => 'BIGSERIAL', NOTNULL => 1,
+            id        => {TYPE => 'INTSERIAL', NOTNULL => 1, 
                           PRIMARYKEY => 1}, 
             bug_id    => {TYPE => 'INT3', NOTNULL => 1,
                           REFERENCES    =>  {TABLE  =>  'bugs',
                                              COLUMN =>  'bug_id',
                                              DELETE => 'CASCADE'}},
-            attach_id => {TYPE => 'INT5',
+            attach_id => {TYPE => 'INT3',
                           REFERENCES    =>  {TABLE  =>  'attachments',
                                             COLUMN  =>  'attach_id',
                                             DELETE => 'CASCADE'}},
@@ -492,7 +492,7 @@ use constant ABSTRACT_SCHEMA => {
 
     attachments => {
         FIELDS => [
-            attach_id    => {TYPE => 'BIGSERIAL', NOTNULL => 1,
+            attach_id    => {TYPE => 'MEDIUMSERIAL', NOTNULL => 1,
                              PRIMARYKEY => 1},
             bug_id       => {TYPE => 'INT3', NOTNULL => 1,
                              REFERENCES    =>  {TABLE  => 'bugs',
@@ -523,7 +523,7 @@ use constant ABSTRACT_SCHEMA => {
     },
     attach_data => {
         FIELDS => [
-            id      => {TYPE => 'INT5', NOTNULL => 1,
+            id      => {TYPE => 'INT3', NOTNULL => 1,
                         PRIMARYKEY => 1,
                         REFERENCES  =>  {TABLE  => 'attachments',
                                          COLUMN => 'attach_id',
@@ -636,7 +636,7 @@ use constant ABSTRACT_SCHEMA => {
                                   REFERENCES => {TABLE  => 'bugs',
                                                  COLUMN => 'bug_id',
                                                  DELETE => 'CASCADE'}},
-            attach_id         => {TYPE => 'INT5',
+            attach_id         => {TYPE => 'INT3',
                                   REFERENCES => {TABLE  => 'attachments',
                                                  COLUMN => 'attach_id',
                                                  DELETE => 'CASCADE'}},
diff --git a/Bugzilla/DB/Schema/Mysql.pm b/Bugzilla/DB/Schema/Mysql.pm
index f2450abf1..5fc50a986 100644
--- a/Bugzilla/DB/Schema/Mysql.pm
+++ b/Bugzilla/DB/Schema/Mysql.pm
@@ -108,12 +108,10 @@ sub _initialize {
         INT2 =>         'smallint',
         INT3 =>         'mediumint',
         INT4 =>         'integer',
-        INT5 =>         'bigint unsigned',
 
         SMALLSERIAL =>  'smallint auto_increment',
         MEDIUMSERIAL => 'mediumint auto_increment',
         INTSERIAL =>    'integer auto_increment',
-        BIGSERIAL =>    'bigint unsigned auto_increment',
 
         TINYTEXT =>     'tinytext',
         MEDIUMTEXT =>   'mediumtext',
diff --git a/Bugzilla/Install/DB.pm b/Bugzilla/Install/DB.pm
index a217e0753..7ab533198 100644
--- a/Bugzilla/Install/DB.pm
+++ b/Bugzilla/Install/DB.pm
@@ -706,14 +706,6 @@ sub update_table_definitions {
     # 2013-08-16 glob@mozilla.com - Bug 905925
     $dbh->bz_add_index('attachments', 'attachments_ispatch_idx', ['ispatch']);
 
-    # 2014-03-11 glob@mozilla.com - Bug 981756 (BMO only)
-    _fix_attachments_primary_key();
-    if ($dbh->bz_column_info('bugs_activity', 'id')->{TYPE} ne 'BIGSERIAL') {
-        $dbh->bz_drop_related_fks('bugs_activity', 'id');
-        $dbh->bz_alter_column('bugs_activity', 'id',
-                              {TYPE => 'BIGSERIAL',  NOTNULL => 1,  PRIMARYKEY => 1});
-    }
-
     ################################################################
     # New --TABLE-- changes should go *** A B O V E *** this point #
     ################################################################
@@ -3745,21 +3737,6 @@ sub _fix_longdescs_primary_key {
     }
 }
 
-sub _fix_attachments_primary_key {
-    my $dbh = Bugzilla->dbh;
-    if ($dbh->bz_column_info('attachments', 'attach_id')->{TYPE} ne 'BIGSERIAL') {
-        $dbh->bz_drop_related_fks('attachments', 'attach_id');
-        $dbh->bz_alter_column('attach_data', 'id', {TYPE => 'INT5'});
-        $dbh->bz_alter_column('bugs_activity', 'attach_id', {TYPE => 'INT5'});
-        $dbh->bz_alter_column('flags', 'attach_id', {TYPE => 'INT5'});
-        # the following two are bmo extensions
-        $dbh->bz_alter_column('flag_state_activity', 'attachment_id', {TYPE => 'INT5'});
-        $dbh->bz_alter_column('autoland_attachments', 'attach_id', {TYPE => 'INT5'});
-        $dbh->bz_alter_column('attachments', 'attach_id',
-                              {TYPE => 'BIGSERIAL',  NOTNULL => 1,  PRIMARYKEY => 1});
-    }
-}
-
 sub _fix_dependencies_dupes {
     my $dbh = Bugzilla->dbh;
     my $blocked_idx = $dbh->bz_index_info('dependencies', 'dependencies_blocked_idx');
diff --git a/extensions/Review/Extension.pm b/extensions/Review/Extension.pm
index 16429ccdd..c22f0a652 100644
--- a/extensions/Review/Extension.pm
+++ b/extensions/Review/Extension.pm
@@ -656,7 +656,7 @@ sub db_schema_abstract_schema {
             },
 
             attachment_id => {
-                TYPE       => 'INT5',
+                TYPE       => 'INT3',
                 REFERENCES => {
                     TABLE  => 'attachments',
                     COLUMN => 'attach_id',
diff --git a/extensions/TryAutoLand/Extension.pm b/extensions/TryAutoLand/Extension.pm
index 8aaadff95..40dbb70d9 100644
--- a/extensions/TryAutoLand/Extension.pm
+++ b/extensions/TryAutoLand/Extension.pm
@@ -61,7 +61,7 @@ sub db_schema_abstract_schema {
     $args->{'schema'}->{'autoland_attachments'} = {
         FIELDS => [
             attach_id => {
-                TYPE       => 'INT5',
+                TYPE       => 'INT3',
                 NOTNULL    => 1,
                 PRIMARYKEY => 1, 
                 REFERENCES => {