diff --git a/zerver/migrations/0244_message_copy_pub_date_to_date_sent.py b/zerver/migrations/0244_message_copy_pub_date_to_date_sent.py index bc31afd5f9..131d3bc1df 100644 --- a/zerver/migrations/0244_message_copy_pub_date_to_date_sent.py +++ b/zerver/migrations/0244_message_copy_pub_date_to_date_sent.py @@ -79,8 +79,19 @@ class Migration(migrations.Migration): """ ), migrations.RunPython(copy_pub_date_to_date_sent, elidable=True), - AddIndexConcurrently( - model_name="message", - index=models.Index("date_sent", name="zerver_message_date_sent_3b5b05d8"), + migrations.SeparateDatabaseAndState( + database_operations=[ + AddIndexConcurrently( + model_name="message", + index=models.Index("date_sent", name="zerver_message_date_sent_3b5b05d8"), + ), + ], + state_operations=[ + migrations.AlterField( + model_name="message", + name="date_sent", + field=models.DateTimeField(db_index=True, null=True, verbose_name="date sent"), + ), + ], ), ] diff --git a/zerver/migrations/0245_message_date_sent_finalize_part1.py b/zerver/migrations/0245_message_date_sent_finalize_part1.py index d3226d9fd3..d5acec928b 100644 --- a/zerver/migrations/0245_message_date_sent_finalize_part1.py +++ b/zerver/migrations/0245_message_date_sent_finalize_part1.py @@ -13,19 +13,16 @@ class Migration(migrations.Migration): """ DROP TRIGGER zerver_message_date_sent_to_pub_date_trigger ON zerver_message; DROP FUNCTION zerver_message_date_sent_to_pub_date_trigger_function(); - - ALTER TABLE zerver_message ALTER COLUMN date_sent SET NOT NULL; - ALTER TABLE zerver_message ALTER COLUMN pub_date DROP NOT NULL; - """, - state_operations=[ - # This just tells Django to, after running the above SQL, consider the AlterField below - # as done. The building of the index actually happened in the previous migration, not here, - # but nevertheless this seems like the correct place to put this fake AlterField. - migrations.AlterField( - model_name="message", - name="date_sent", - field=models.DateTimeField(db_index=True, verbose_name="date sent"), - ), - ], + """ + ), + migrations.AlterField( + model_name="message", + name="date_sent", + field=models.DateTimeField(db_index=True, verbose_name="date sent"), + ), + migrations.AlterField( + model_name="message", + name="pub_date", + field=models.DateTimeField(db_index=True, null=True, verbose_name="date published"), ), ] diff --git a/zerver/models.py b/zerver/models.py index 90517904a2..7fedcb89bf 100644 --- a/zerver/models.py +++ b/zerver/models.py @@ -3104,7 +3104,6 @@ class Message(AbstractMessage): indexes = [ GinIndex("search_tsvector", fastupdate=False, name="zerver_message_search_tsvector"), models.Index(Upper("subject"), name="upper_subject_idx"), - models.Index("date_sent", name="zerver_message_date_sent_3b5b05d8"), models.Index( "recipient", Upper("subject"),