kotovalexarian-likes-gitlab/gitlab-org--gitlab-foss
1
0
Fork 0
Code Releases Activity

Update migration comment

Browse source 
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
This commit is contained in:
Dmitriy Zaporozhets 2016-03-30 10:38:03 +02:00
parent 065faac3a3
commit 1a293a4384
1 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
db/migrate/20160328115649_migrate_new_notification_setting.rb
View file

@ -1,6 +1,10 @@
# This migration will create one row of NotificationSetting for each Member row # This migration will create one row of NotificationSetting for each Member row
# It can take long time on big instances. Its unclear yet if this migration can be done online. # It can take long time on big instances.
# This comment should be updated by @dzaporozhets before 8.7 release. If not - please ask him to do so. #
# This migration can be done online but with following effects:
# - during migration some users will receive notifications based on their global settings (project/group settings will be ignored)
# - its possible to get duplicate records for notification settings since we don't create uniq index yet
#
class MigrateNewNotificationSetting < ActiveRecord::Migration class MigrateNewNotificationSetting < ActiveRecord::Migration
def up def up
timestamp = Time.now timestamp = Time.now