forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit97e971e
committed
Fix determination when slot types for upper executor nodes are fixed.
For many queries the fact that the tuple descriptor from the lowernode was not taken into account when determining whether the type of aslot is fixed, lead to tuple deforming for such upper nodes not to beJIT accelerated.I broke this in675af5c.There is ongoing work to enable writing regression tests for relatedbehavior (including a patch that would have detected thisregression), by optionally showing such details in EXPLAIN. But as itseems unlikely that that will be suitable for stable branches, justmerge the fix for now.While it's fairly close to the 12 release window, the fact that 11continues to perform JITed tuple deforming in these cases, thatthere's still cases where we do so in 12, and the fact that theperformance regression can be sizable, weigh in favor of fixing itnow.Author: Andres FreundDiscussion:https://postgr.es/m/20190927072053.njf6prdl3vb7y7qb@alap3.anarazel.deBackpatch: 12-, where675af5c was merged.1 parent258bf86 commit97e971e
1 file changed
+2
-0
lines changedLines changed: 2 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2395 | 2395 |
| |
2396 | 2396 |
| |
2397 | 2397 |
| |
| 2398 | + | |
2398 | 2399 |
| |
2399 | 2400 |
| |
2400 | 2401 |
| |
| |||
2414 | 2415 |
| |
2415 | 2416 |
| |
2416 | 2417 |
| |
| 2418 | + | |
2417 | 2419 |
| |
2418 | 2420 |
| |
2419 | 2421 |
| |
|
0 commit comments
Comments
(0)