django orm: select_related, fooling reverse foreign key with a fake foreign key addition to the model, what can go wrong?

这一生的挚爱 提交于 2020-01-24 15:06:13

问题


I am trying to learn how to use Django's ORM for more advanced queries, instead of using raw sql.

select_related makes joins to reduce database hits, in principle it could make the joins that I would do manually.

But there is a problem: It doesn't use reverse foreign key relationships to make the sql. For the schema I have this is a nuisance. I have found what looks like a surprisingly easy work around, and I'm worried that it will be incorrect in ways I don't understand.

I'm using a legacy DB, and it is read-only, and therefore unmanaged by the ORM, and there are no consequences for cascade settings. I have made models for its tables via manage.py analysedb which did a very good job; I have a 6000 line model file, and only needed to spend ten minutes fixing things.

One tweak I did was a hack to fool select_related by creating a foreign key relationship which is actually a reverse FK relationship. This means I have two entries in my model pointing to the same database column, which is the primary key (correct) and also now a foreign key. I exploit the "new" column in select_related.

The main table for my query is Service. It has foreign key relationships to a number of other tables, and that works well with select_related.

The column servrecid_1 is my artificial addition. The other two columns are two of the genuine FKs; there are more.

class Service(models.Model):
    servrecid = models.AutoField(db_column='ServRecID', primary_key=True)
    servrecid_1 = models.ForeignKey('RecServ', models.DO_NOTHING, to_field='servrecid',
                                    db_column='ServRecID')  # Field name made lowercase.

    visitrecordid = models.ForeignKey('Visit', models.DO_NOTHING, db_column='VisitRecordID', blank=True,
                                      null=True)  # Field name made lowercase.
    itemno = models.ForeignKey(Fees, models.DO_NOTHING, db_column='ItemNo', to_field='itemno')
    ...
    class Meta:
        managed = False  # same for all the models


class RecServ(models.Model):
    allocationid = models.AutoField(db_column='AllocationID', primary_key=True)  # Field name made lowercase.
    servrecid = models.ForeignKey('Service', models.DO_NOTHING, db_column='ServRecID')  # Field name made lowercase.
    receiptno = models.ForeignKey(Receipt, models.DO_NOTHING, db_column='ReceiptNo')  # Field name made lowercase.

(There are more relations than I show in the snippets above)

With this, I can now do queries like this:

q = models.Service.objects.select_related('visitrecordid__servdoctor').select_related('visitrecordid__invoiceto') \
        .select_related('visitrecordid__patientno').select_related('itemno').select_related(
    'servrecid_1__receiptno').all()[:5]

which creates a query with these joins:

    ... FROM [SERVICE] INNER JOIN [REC_SERV] ON ([SERVICE].[ServRecID] = [REC_SERV].[ServRecID])
 INNER JOIN [RECEIPT] ON ([REC_SERV].[ReceiptNo] = [RECEIPT].[ReceiptNo]) 
LEFT OUTER JOIN [VISIT] ON ([SERVICE].[VisitRecordID] = [VISIT].[VisitRecordID]) 
LEFT OUTER JOIN [CM_PATIENT] ON ([VISIT].[PatientNo] = [CM_PATIENT].[PATIENT_ID]) 
LEFT OUTER JOIN [DOCTOR] ON ([VISIT].[ServDoctor] = [DOCTOR].[DoctorCode]) 
LEFT OUTER JOIN [INVOICETO] ON ([VISIT].[InvoiceTo] = [INVOICETO].[InvoiceTo]) 
INNER JOIN [FEES] ON ([SERVICE].[ItemNo] = [FEES].[ItemNo])

The first join only appears because of my false FK. The SQL looks fine to me, I think I have solved my problem.

Should this actually work? What will happen now with this mutual foreign key relationship between both tables?

来源:https://stackoverflow.com/questions/59124549/django-orm-select-related-fooling-reverse-foreign-key-with-a-fake-foreign-key

易学教程内所有资源均来自网络或用户发布的内容,如有违反法律规定的内容欢迎反馈
该文章没有解决你所遇到的问题?点击提问,说说你的问题,让更多的人一起探讨吧!