Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

Add exists filter syntax

Ouverte
  • Quelque chose ne va pas ?
#2,002 2 commentaires 1 réaction 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 2 jours

Personne n'a encore pris cette issue.

Évaluation

Difficulté
5/5
Temps estimé
Plus d'une semaine
Accessibilité débutants
42/100
Type d'issue
Fonctionnalité
Clarté
Plutôt claire
Activité
Calme
Stack technique
python, sql
Domaine
databases

Piste de recherche

Commencez par l’issue connexe #385 et comparez la syntaxe de filtre proposée avec les exemples SQL EXISTS et JOIN de cette issue. Clarifiez où __exists doit se trouver et quelle devrait être sa sémantique ; le travail est considéré comme terminé lorsqu’une syntaxe acceptée génère un filtrage basé sur l’existence sans nécessiter distinct().

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Is your feature request related to a problem? Please describe.

When querying for the existence of a one-to-many related model with certain values it one of its fields, the .distinct() method is required to prevent duplicates. This is not the most efficient query.

Contrived example: I have to do this to find all phone numbers of users that work at the north pole without duplicates:

PhoneNumber.filter(user__company__location='North Pole').distinct()

According to ChatGPT,

SELECT m.*
FROM model m
WHERE EXISTS (
    SELECT 1
    FROM user u
    JOIN company c ON u.company_id = c.id
    WHERE u.id = m.user_id
      AND c.location = 'North Pole'
);

is preferable over

SELECT DISTINCT m.*
FROM model m
LEFT OUTER JOIN user u ON m.user_id = u.id
LEFT OUTER JOIN company c ON u.company_id = c.id
WHERE c.location = 'North Pole';

Why this is better (copied from ChatGPT):

  • EXISTS checks for existence rather than joining, so no duplicates.
  • Often more efficient, since the DB can stop searching after the first match.
  • Cleaner semantics: “Does this related row exist?” rather than “Give me all the related rows, then deduplicate.”

Describe the solution you'd like

PhoneNumber.filter(user__company__exists__location='North Pole')

Describe alternatives you've considered

Could be variations on the placement of __exists in the query. I realize the exact semantics could be a bit unclear.

Additional context

Related #385

Langage dominant
Python
Étoiles
5.6k
Forks
523
Merge moyen
2 j 18 h
PR mergées (30 j)
12

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de tortoise/tortoise-orm

Toutes les issues de tortoise/tortoise-orm

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.