Admin : Éviter les doublons lorsqu’on liste les PASS IAE d’une entreprise

[francoisfreitag]

🤔 Pourquoi ?

Si un candidat avait émit plusieurs candidatures vers une entreprise, son PASS était listé une fois par candidature.

🍰 Comment ?

Exists

🏝️ Comment tester

Comparer /admin/approvals/approval/?assigned_company=11919&o=2.-6 entre la prod et cette branche.

[rsebille]

❤️

[rsebille]

De ce que je comprend ça va faire la sous-requête pour chaque ligne donc dans le cas présent ça serait pas plus efficace d'utiliser d'un IN ?

Suggested change

	self.filter(Exists(JobApplication.objects.filter(approval=OuterRef("pk"), to_company=company_id)))
	self.filter(pk__in=Subquery(JobApplication.objects.filter(to_company=company_id).values("approval")))

[francoisfreitag]

Et bien... c’est équivalent 😂
Le EXPLAIN de DjDT donne le même plan d’exécution, peu importe la forme. J’ai inclus la requête dans les deux cas, où on voit bien soit le IN soit le EXISTS(). Je suppose que PostgreSQL est malin et prend l’opération la plus efficace pour réaliser la requête, peu importe la forme demandée.

exists.sql.gz
in.sql.gz

J’ai préféré ta version qui me semble effectivement plus simple 👍

[rsebille]

Yep mais à mon avis ça dépend aussi de ce que tu lui demandes à coté, là c'est probablement lié à l'annotation assigned_company qui tape sur la même table et donc il optimise le travail, sur un truc plus simple c'est pas dit.