forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitc6c3b3b
committed
Remove gratuitous assumptions about what make_modifytable can see.
For no clearly good reason, make_modifytable assumed that itcould not reach its get-the-FDW-info-the-hard-way path in MERGE.It's currently possible to demonstrate that assertion failing,which seems to be due to an upstream planner bug; but there's nogood reason to do it like this at all. Let's apply the principleof separation of concerns and make the MERGE check separately,after getting or not getting the fdwroutine pointer.Per report from Alexander Lakhin. No test case, since I thinkthe potential test condition will go away soon.Discussion:https://postgr.es/m/36bee393-b351-16ac-93b2-d46d83637e45@gmail.com1 parenta316a3b commitc6c3b3b
1 file changed
+20
-22
lines changedLines changed: 20 additions & 22 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
7093 | 7093 |
| |
7094 | 7094 |
| |
7095 | 7095 |
| |
7096 |
| - | |
7097 |
| - | |
7098 |
| - | |
7099 |
| - | |
7100 |
| - | |
7101 |
| - | |
7102 |
| - | |
7103 |
| - | |
7104 |
| - | |
7105 |
| - | |
7106 |
| - | |
7107 |
| - | |
7108 |
| - | |
7109 |
| - | |
7110 |
| - | |
7111 |
| - | |
7112 |
| - | |
7113 |
| - | |
7114 |
| - | |
7115 | 7096 |
| |
7116 | 7097 |
| |
7117 | 7098 |
| |
7118 | 7099 |
| |
7119 | 7100 |
| |
7120 |
| - | |
7121 |
| - | |
7122 |
| - | |
| 7101 | + | |
| 7102 | + | |
7123 | 7103 |
| |
7124 | 7104 |
| |
7125 | 7105 |
| |
7126 | 7106 |
| |
7127 | 7107 |
| |
| 7108 | + | |
| 7109 | + | |
| 7110 | + | |
| 7111 | + | |
| 7112 | + | |
| 7113 | + | |
| 7114 | + | |
| 7115 | + | |
| 7116 | + | |
| 7117 | + | |
| 7118 | + | |
| 7119 | + | |
| 7120 | + | |
| 7121 | + | |
| 7122 | + | |
| 7123 | + | |
| 7124 | + | |
| 7125 | + | |
7128 | 7126 |
| |
7129 | 7127 |
| |
7130 | 7128 |
| |
|
0 commit comments
Comments
(0)