Commit ee3a551
committed
Fix incorrect logic in plan dependency recording
Both 50e17ad and 29f45e2 mistakenly tried to record a plan dependency
on a function but mistakenly inverted the OidIsValid test. This meant
that we'd record a dependency only when the function's Oid was
InvalidOid. Clearly this was meant to *not* record the dependency in
that case.
50e17ad made this mistake first, then in v15 29f45e2 copied the same
mistake.
Reported-by: Tom Lane
Backpatch-through: 14, where 50e17ad first made this mistake
Discussion: https://postgr.es/m/2277537.1694301772@sss.pgh.pa.us1 parent f062cdd commit ee3a551Copy full SHA for ee3a551
File tree
Expand file treeCollapse file tree
1 file changed
+2
-2
lines changedOpen diff view settings
Filter options
- src/backend/optimizer/plan
Expand file treeCollapse file tree
1 file changed
+2
-2
lines changedOpen diff view settings
Collapse file
src/backend/optimizer/plan/setrefs.c
Copy file name to clipboardExpand all lines: src/backend/optimizer/plan/setrefs.c+2-2Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
1991 | 1991 | |
1992 | 1992 | |
1993 | 1993 | |
1994 | | - |
| 1994 | + |
1995 | 1995 | |
1996 | 1996 | |
1997 | | - |
| 1997 | + |
1998 | 1998 | |
1999 | 1999 | |
2000 | 2000 | |
|
0 commit comments