paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-02 13:03:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

messages: Clarify comment on zerver_message_edit_history_id idx.

Browse Source 
This commit removes a misleading comment regarding
'zerver_message_edit_history_id' index.

We added the index in 0679 to use in 0680 but later the 0680 migration
was reworked resulting in the index not being used in 0680.

We didn't drop the index as we expect it to be helpful for other
things.

The comment was misleading hence removed.
This commit is contained in:
Prakhar Pratyush
2025-03-28 10:35:59 +05:30
committed by Tim Abbott
parent e1aa8b1cb0
commit 68b7deac72
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
zerver/models/messages.py
View File

@@ -244,7 +244,11 @@ class Message(AbstractMessage):
name="zerver_message_realm_id",
),
models.Index(
# Used by 0680_rename_general_chat_to_empty_string_topic
# Potentially useful for migrations that rewrite
# message edit history. Originally added for
# 0680_rename_general_chat_to_empty_string_topic,
# though that migration was adjusted in a way that no
# longer uses this.
fields=["id"],
condition=Q(edit_history__isnull=False),
name="zerver_message_edit_history_id",