- Notifications
You must be signed in to change notification settings - Fork5
Commite54b10a
committed
Remove the "last ditch" code path in join_search_one_level().
So far as I can tell, it is no longer possible for this heuristic to doanything useful, because the new weaker definition ofhave_relevant_joinclause means that any relation with a joinclause must beconsidered joinable to at least one other relation. It would still bepossible for the code block to be entered, for example if there are joinorder restrictions that prevent any join of the current level from beingformed; but in that case it's just a waste of cycles to attempt to formcartesian joins, since the restrictions will still apply.Furthermore, IMO the existence of this code path can mask bugs elsewhere;we would have noticed the problem with cartesian joins a lot sooner ifthis code hadn't compensated for it in the simplest case.Accordingly, let's remove it and see what happens. I'm committing thisseparately from the prerequisite changes in have_relevant_joinclause,just to make the question easier to revisit if there is some fault inmy logic.1 parente3ffd05 commite54b10a
1 file changed
+17
-59
lines changedLines changed: 17 additions & 59 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
169 | 169 |
| |
170 | 170 |
| |
171 | 171 |
| |
172 |
| - | |
173 |
| - | |
174 |
| - | |
175 |
| - | |
176 |
| - | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
177 | 180 |
| |
178 |
| - | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
179 | 185 |
| |
180 |
| - | |
181 |
| - | |
182 |
| - | |
| 186 | + | |
| 187 | + | |
| 188 | + | |
183 | 189 |
| |
184 |
| - | |
185 |
| - | |
186 |
| - | |
187 |
| - | |
188 |
| - | |
189 |
| - | |
190 |
| - | |
191 |
| - | |
192 |
| - | |
193 |
| - | |
194 |
| - | |
195 |
| - | |
196 |
| - | |
197 |
| - | |
198 |
| - | |
199 |
| - | |
200 |
| - | |
201 |
| - | |
202 |
| - | |
203 |
| - | |
204 |
| - | |
205 |
| - | |
206 |
| - | |
207 |
| - | |
208 |
| - | |
209 |
| - | |
210 |
| - | |
211 |
| - | |
212 |
| - | |
213 |
| - | |
214 |
| - | |
215 |
| - | |
216 |
| - | |
217 |
| - | |
218 |
| - | |
219 |
| - | |
220 |
| - | |
221 |
| - | |
222 |
| - | |
223 |
| - | |
224 |
| - | |
225 |
| - | |
226 |
| - | |
| 190 | + | |
| 191 | + | |
227 | 192 |
| |
228 | 193 |
| |
229 | 194 |
| |
| |||
751 | 716 |
| |
752 | 717 |
| |
753 | 718 |
| |
754 |
| - | |
755 |
| - | |
756 |
| - | |
757 |
| - | |
758 |
| - | |
759 |
| - | |
760 |
| - | |
761 | 719 |
| |
762 | 720 |
| |
763 | 721 |
| |
|
0 commit comments
Comments
(0)