Update _id field before .save() for ForeignKey

28 مرّة مشاهدة
التخطي إلى أول رسالة غير مقروءة

Maxim Bulatov

غير مقروءة،
29‏/11‏/2019، 7:42:00 ص29‏/11‏/2019
إلى Django users
Hello,

I have lot of lists like follow:
(Name1): (V1, V2, V3, V4, ..)

And I use many threads to create many db objects:
company = Company(Name)
for x in vector:
    v = CompanyValue(company=company, value=x)
    values.append(v)

class CompanyValue(models.Model):
    company = models.ForeignKey('Company', on_delete=models.CASCADE)
    value = FloatField

This pool of threads return all lists, what I combine, filter and want put to database in one bulk_create call.
Suddenly, I found, company.id is not ready to be linked in CompanyValue, follow code does not works:
Company.objects.bulk_create(companies)
CompanyValue.objects.bulk_create(values)      # value.company_id is null here

No way to save objects in my threads, because I need to filter objects and can do it only when all of them are collected. Also, it has performance issues 100k+ of requests is slowly enough. I have two ways to resolve:
1. I can make intermediate class or tuple and create CompanyValue objects only when companies is saved to db. Hard to support, hard to filter, not so clean.
2. I can use dirty hack to update _id field:
    for v in values:
        v.company = v.company

Can you advice more ways for me? It seems, I don't know something from django features to make it elegant.
I see 11 years old issue here (https://code.djangoproject.com/ticket/9553) and it marked as intended behaviour.

Integr@te System

غير مقروءة،
29‏/11‏/2019، 11:00:12 ص29‏/11‏/2019
إلى django...@googlegroups.com
Hi guy,

I see this doc fix for u, with builtin pk in models django.
And you can create index in text/char field to optimize performance.



--
You received this message because you are subscribed to the Google Groups "Django users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to django-users...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/django-users/3d62beaf-7fbf-492f-b998-1539f45cd7a0%40googlegroups.com.

Dvenum

غير مقروءة،
29‏/11‏/2019، 2:28:20 م29‏/11‏/2019
إلى Django Users
Integr@te System, thanks. If I set pk (id) manually, it works, but hard to specify this id value. Maybe for one thread this way is good.



--

Charles Lee

غير مقروءة،
29‏/11‏/2019، 7:51:40 م29‏/11‏/2019
إلى Django users
I have an idea.
  1. `bulk_create` all companies.
  2. `Filter`(SELECT) all `Company` and make it dictionary for retrieving company fast.
    1. company_dict = {
        Name’: ‘pk’,
        ...
      }
  3. Create a list of `CompanyValue` with company name and company_dict.
  4. `bulk_create` the list.

I'm not sure if it will work because I haven't implemented it, but I hope it helps. 

2019년 11월 29일 금요일 오후 9시 42분 0초 UTC+9, Maxim Bulatov 님의 말:
تم حذف الرسالة.

Dvenum

غير مقروءة،
30‏/11‏/2019، 1:31:21 ص30‏/11‏/2019
إلى Django Users
Interest idea and it works, but not necessary:
    company = Company()
    #company.id is None
    companies.append(company)
    Company.objects.bulk_create(companies)
    #company.id is 1

Instead to ask pk from database, it can be used from company_value.company.id. Django save pk value to internal meta (models/base.py) and does not update it. This works too:
value.company = value.company     # pk updated

I want to ask, each child object should be created only when parent is saved to db? Maybe django has something to cover this case?


--
You received this message because you are subscribed to the Google Groups "Django users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to django-users...@googlegroups.com.


--

Integr@te System

غير مقروءة،
30‏/11‏/2019، 2:39:28 ص30‏/11‏/2019
إلى django...@googlegroups.com
Hi Dvenun,

Bc of  Django implement accord to python, of cause child inherit from parent existence.

Dvenum

غير مقروءة،
30‏/11‏/2019، 3:40:20 ص30‏/11‏/2019
إلى Django Users
Integr@te System,

This is True: id(company) == id(company_value.company)
So, we have exact the same object company, .id is ready on .save() call, but internal meta dictionary still has None, because it was none when company_value created.



--

Integr@te System

غير مقروءة،
30‏/11‏/2019، 6:27:47 ص30‏/11‏/2019
إلى django...@googlegroups.com

Dvenum

غير مقروءة،
30‏/11‏/2019، 9:29:43 ص30‏/11‏/2019
إلى Django Users
Thanks, its interest to know more about pk management, but this is another case. Now I update pk for child objects and this is fastest way, what I found. If django will reread .pk before save, it will the same.

Want to know, maybe my logic can be altered from the start. How peoples create batch of objects with relations to insert it in one request? I think, this is popular task.




--
الرد على الكل
رد على الكاتب
إعادة توجيه
0 رسالة جديدة