Skip to content

Cleanup bigint conversion for ci_builds

Krasimir Angelov requested to merge 338067-ci-builds-cleanup-bigint-conversion into master

What does this MR do and why?

This MR removes columns and triggers used for the bigint conversion of table ci_builds.

  • remove column id_convert_to_bigint
  • remove column stage_id_convert_to_bigint
  • remove trigger_3f6129be01d2 trigger and function

Related to #338067 (closed).

Database migrations

Up

$ bundle exec rails db:migrate:up VERSION=20210915022415
== 20210915022415 CleanupBigintConversionForCiBuilds: migrating ===============
-- remove_column(:ci_builds, "id_convert_to_bigint")
   -> 0.0013s
-- remove_column(:ci_builds, "stage_id_convert_to_bigint")
   -> 0.0009s
== 20210915022415 CleanupBigintConversionForCiBuilds: migrated (0.0231s) ======

Down

$ bundle exec rails db:migrate:down VERSION=20210915022415
== 20210915022415 CleanupBigintConversionForCiBuilds: reverting ===============
-- table_exists?(:ci_builds)
   -> 0.0004s
-- column_exists?(:ci_builds, :id)
   -> 0.0022s
-- column_exists?(:ci_builds, :id)
   -> 0.0018s
-- column_exists?(:ci_builds, :stage_id)
   -> 0.0018s
-- columns(:ci_builds)
   -> 0.0017s
-- add_column(:ci_builds, "id_convert_to_bigint", :int, {:default=>0, :null=>false})
   -> 0.0030s
-- columns(:ci_builds)
   -> 0.0017s
-- add_column(:ci_builds, "stage_id_convert_to_bigint", :int, {:default=>nil})
   -> 0.0009s
== 20210915022415 CleanupBigintConversionForCiBuilds: reverted (0.0371s) ======

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports