#124 Retour sur d711e9c740a7fda98e3049d9ee5322e631363fc3 et fix #4465

Open
cerdic wants to merge 1 commits from issue_4465 into master
cerdic commented 1 week ago
Owner

l'absence de clause where sur les jointures objet/id_objet permet en effet d'optimiser les boucles de type
<BOUCLE(ARTICLES){id_document ?}>
mais a contrario comme indique par #4465 l'optimisation d'une boucle
`<BOUCLE(DEPOTS){id_mot=1}>#ID_DEPOT est trop forte car la jointure saute du fait que la table principale est enlevee du from et on a plus le where sur objet.

On s'en tire on reinserant de nouveau le where en doublon de la clause de jointure,
mais en le pointant d'une cle qui l'associe a la jointure,
ce qui permet a calculer_select de savoir que ce where saute si la jointure saute et de faire correctement son travail dans les 2 cas

Il faut esperer que ca ne nous cree pas un nouveau cas limite, ou pas avant 12 ans en tout cas :p

l'absence de clause where sur les jointures objet/id_objet permet en effet d'optimiser les boucles de type `<BOUCLE(ARTICLES){id_document ?}>` mais a contrario comme indique par #4465 l'optimisation d'une boucle `<BOUCLE(DEPOTS){id_mot=1}>#ID_DEPOT</BOUCLE> est trop forte car la jointure saute du fait que la table principale est enlevee du from et on a plus le where sur objet. On s'en tire on reinserant de nouveau le where en doublon de la clause de jointure, mais en le pointant d'une cle qui l'associe a la jointure, ce qui permet a calculer_select de savoir que ce where saute si la jointure saute et de faire correctement son travail dans les 2 cas Il faut esperer que ca ne nous cree pas un nouveau cas limite, ou pas avant 12 ans en tout cas :p
cerdic added 1 commit 1 week ago
3828abd23c Retour sur d711e9c740 et fix #4465 :
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.