Development #38691
supprimer le CROSS JOIN lors des jointures
Début:
23 décembre 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Il cache des valeurs (NULL,x) ou (x,NULL). C'était une mauvaise idée due au besoin d'obtenir la totalité des valeurs croisées (mais il manque donc les valeurs nulles, maintenant que c'est corrigé via la récupération des énumérations autrement on en a plus besoin).
Fichiers
Révisions associées
engine: remove use of cross join (#38691)
It prevented reporting of NULL values.
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0002-engine-remove-use-of-cross-join-38691.patch 0002-engine-remove-use-of-cross-join-38691.patch ajouté
- Fichier 0001-engine-simplifiy-equivalent-code-38691.patch 0001-engine-simplifiy-equivalent-code-38691.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Pour faire joli tu peux remplacer le
for kind in ['left', 'inner', 'right', 'full']:
par
for kind in JOIN_KINDS:
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 16b60621d4cf1a1f8576cec28eb59e9bf75627cd Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Mon Jan 6 22:07:05 2020 +0100 engine: remove use of cross join (#38691) It prevented reporting of NULL values. commit 8f204362a44319d6ce46da9f313f9a46137fd7fb Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Fri Jan 3 11:09:28 2020 +0100 engine: simplifiy equivalent code (#38691) The two branches built the same string.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
Thomas Noël a écrit :
Pour faire joli tu peux remplacer le
for kind in ['left', 'inner', 'right', 'full']:
par
for kind in JOIN_KINDS:
Oui mais non parce que j'avais très envie que ce soit fait dans cet ordre et avant python 3.jenesaisplus les dictionnaires de base ne sont toujours par ordonnés.
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
engine: simplifiy equivalent code (#38691)
The two branches built the same string.