Django admin list_display 使用外键时异常缓慢

Question

姜戈 1.2.5 Python：2.5.5

我的运动模型管理列表速度非常慢（400 条记录需要 5 分钟）。它在一秒钟左右就恢复了，直到我们得到了 400 场比赛、50 支球队和 2 项运动。

我已经以一种糟糕的方式修复了它，所以我想看看是否有人以前见过这个。我的应用程序如下所示：

models:

Sport( models.Model )
    name

Venue( models.Model )
    name

Team( models.Model )
    name

Fixture( models.Model )
    date
    sport = models.ForeignKey(Sport)
    venue = models.ForeignKey(Venue)

TeamFixture( Fixture )
    team1 = models.ForeignKey(Team, related_name="Team 1")
    team2 = models.ForeignKey(Team, related_name="Team 2")


admin:

TeamFixture_ModelAdmin (ModelAdmin)
    list_display = ('date','sport','venue','team1','team2',)

如果我从 list_display 中删除任何外键，那么速度很快。一旦我添加任何外键，速度就会变慢。

我通过使用非外键但在模型初始化中计算它们来修复它，这样就可以了：

models:

TeamFixture( Fixture )
    team1 = models.ForeignKey(Team, related_name="Team 1")
    team2 = models.ForeignKey(Team, related_name="Team 2")
    sport_name = ""
    venue_name = ""
    team1_name = ""
    team2_name = ""

    def __init__(self, *args, **kwargs):
        super(TeamFixture, self).__init__(*args, **kwargs)

        self.sport_name = self.sport.name
        self.venue_name = self.venue.name
        self.team1_name = self.team1.name
        self.team2_name = self.team2.name

admin:

TeamFixture_ModelAdmin (ModelAdmin)
    list_display = ('date','sport_name','venue_name','team1_name','team2_name',)

所有其他模型的管理都很好，目前有几千条记录，并且实际站点中的所有视图都运行良好。

Answer 1

这让我发疯。 list_select_lated 设置为 True，但是在 list_display 中向 User 添加外键会在管理中的每一行生成一个查询，这会使列表变慢。 Select_lated 为 True，因此 Django 管理员不应在每一行上调用此查询。
到底是怎么回事 ？

Answer 2

我首先要寻找的是数据库调用。如果您不应该这样做，请安装 django-debug-toolbar。这个很棒的工具可以让您检查为当前请求完成的所有 SQL 查询。我想他们有很多。如果您查看它们，您就会知道在哪里寻找问题。

我自己也遇到过一个问题：当模型的 __unicode__ 方法使用外键时，会导致每个实例一次数据库命中。我知道有两种方法可以解决这个问题：

• 使用 select_lated，这通常是您最好的选择。
• 让您的 __unicode__ 返回静态字符串并重写 save 方法以相应地更新此字符串。

Answer 3

这是 django admin 和外键的一个非常老的问题。这里发生的情况是，每当您尝试加载一个对象时，它都会尝试获取该外键的所有对象。假设您正在尝试加载一些球队的赛程（假设球队数量约为 100 支），它将继续一次性包含所有 100 支球队。您可以尝试使用 raw_fields 来优化它们。这样做的目的是，不必一次调用所有内容，而是限制调用次数，并确保仅在触发事件时（即，当您选择团队时）才进行调用。
如果这看起来有点像 UI 混乱，您可以尝试使用此类：

"""
For Raw_id_field to optimize django performance for many to many fields
"""
class RawIdWidget(ManyToManyRawIdWidget):
    def label_for_value(self, value):
        values = value.split(',')
        str_values = []
        key = self.rel.get_related_field().name
        for v in values:
            try:
                obj = self.rel.to._default_manager.using(self.db).get(**{key: v})
                x = smart_unicode(obj)
                change_url = reverse(
                    "admin:%s_%s_change" % (obj._meta.app_label, obj._meta.object_name.lower()),
                    args=(obj.pk,)
                )
                str_values += ['<strong><a href="%s">%s</a></strong>' % (change_url, escape(x))]
            except self.rel.to.DoesNotExist:
                str_values += [u'No input or index in the db']
        return u', '.join(str_values)

class ImproveRawId(admin.ModelAdmin):
    raw_id_fields = ('created_by', 'updated_by')
    def formfield_for_dbfield(self, db_field, **kwargs):
        if db_field.name in self.raw_id_fields:
            kwargs.pop("request", None)
            type = db_field.rel.__class__.__name__
            kwargs['widget'] = RawIdWidget(db_field.rel, site)
            return db_field.formfield(**kwargs)
        return super(ImproveRawId, self).formfield_for_dbfield(db_field, **kwargs)

只需确保正确继承该类即可。我猜测类似TeamFixture_ModelAdmin (ImproveRawIdFieldsForm)。这很可能会给你的 django 管理带来相当酷的性能提升。

Answer 4

我通过将 list_select_lated 设置为相关模型字段列表而不是仅 True 解决了我的问题