forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit17b715c
committed
Add test case for EEOP_INNER_SYSVAR/EEOP_OUTER_SYSVAR executor opcodes.
The EEOP_INNER_SYSVAR and EEOP_OUTER_SYSVAR executor opcodes are notexercised by normal queries, because setrefs.c will resolve the referencesto system columns in the scan nodes already. Join nodes refer to them bytheir position in the child node's target list, like user columns.The only place where those opcodes are used, is in evaluating a trigger'sWHEN condition that references system columns. Trigger evaluation abusesthe INNER/OUTER Vars to refer to the OLD and NEW tuples. The code to handlethe opcodes is pretty straightforward, but it seems like a good idea tohave some test coverage for them, anyway, so that they don't get removed orbroken by accident.Author: Ashutosh Bapat, with some changes by me.Discussion:https://www.postgresql.org/message-id/CAFjFpRerUFX=T0nSnCoroXAJMoo-xah9J+pi7+xDUx86PtQmew@mail.gmail.com1 parent1486f7f commit17b715c
2 files changed
+20
-0
lines changedLines changed: 10 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
440 | 440 |
| |
441 | 441 |
| |
442 | 442 |
| |
| 443 | + | |
| 444 | + | |
| 445 | + | |
| 446 | + | |
| 447 | + | |
| 448 | + | |
| 449 | + | |
| 450 | + | |
| 451 | + | |
| 452 | + | |
443 | 453 |
| |
444 | 454 |
| |
445 | 455 |
| |
|
Lines changed: 10 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
291 | 291 |
| |
292 | 292 |
| |
293 | 293 |
| |
| 294 | + | |
| 295 | + | |
| 296 | + | |
| 297 | + | |
| 298 | + | |
| 299 | + | |
| 300 | + | |
| 301 | + | |
| 302 | + | |
| 303 | + | |
294 | 304 |
| |
295 | 305 |
| |
296 | 306 |
| |
|
0 commit comments
Comments
(0)