Django快速删除除最近1000条外所有指定对象的实现方法

报错原因说明 #

你之前的两种写法报错都是Django QuerySet的底层限制：

• 调用last()报错：QuerySet执行切片[:1000]后对应SQL的LIMIT子句，而last()本质是先反转QuerySet再取首条，Django不允许对已切片的QuerySet执行反转操作。
• 负索引报错：Django QuerySet不支持负索引，因为SQL本身没有负偏移量的语法，无法通过[-1]取切片的最后一个元素。

最优实现方案 #

核心思路是先定位需要保留的最早的那条Event的边界，再直接删除所有早于边界的记录，仅需要3次数据库查询，避免了传递1000个ID到IN子句的低效率操作。

前置优化 #

先给Event表添加user和created的联合索引，过滤和排序都会直接走索引，性能提升最明显：

# 对应Model的Meta定义示例
class Event(models.Model):
    user = models.ForeignKey(User, on_delete=models.CASCADE)
    created = models.DateTimeField(auto_now_add=True)
    # 其他字段...

    class Meta:
        indexes = [
            models.Index(fields=['user', '-created']),
        ]

代码实现 #

from django.db.models import Q

user_events = Event.objects.filter(user=user).order_by('-created', '-id')
event_count = user_events.count()

if event_count > 1000:
    # 取第1000条需要保留的边界事件，索引从0开始所以取999
    boundary_event = user_events[999]
    # 一次性删除所有早于边界的记录，同时处理created时间相同的场景
    Event.objects.filter(
        user=user,
        Q(created__lt=boundary_event.created) | Q(created=boundary_event.created, id__lt=boundary_event.id)
    ).delete()

精简写法（仅适用于自增ID和创建时间正相关的场景） #

如果你的业务中Event的创建时间和自增ID完全正相关（新生成的Event ID一定更大），可以简化为：

user_events = Event.objects.filter(user=user).order_by('-id')
if user_events.count() > 1000:
    boundary_id = user_events.values_list('id', flat=True)[999]
    Event.objects.filter(user=user, id__lt=boundary_id).delete()

执行效率更高，不需要额外处理时间重复的问题。

内容的提问来源于stack exchange，提问作者Alex Zaitsev