forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit1585ff7
committed
Don't allow GetTransactionSnapshot() in logical decoding
A historic snapshot should only be used for catalog access, notgeneral queries. We never call GetTransactionSnapshot() during logicaldecoding, which is good because it wouldn't be very sensible, so thecode to deal with that was unreachable and untested. Turn it into anerror, to avoid doing that in the future either.Discussion:https://www.postgresql.org/message-id/a868fe78-ddb4-4b0a-9b96-873d91d93cfd@iki.fi1 parent952365c commit1585ff7
1 file changed
+4
-8
lines changedLines changed: 4 additions & 8 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
212 | 212 |
| |
213 | 213 |
| |
214 | 214 |
| |
215 |
| - | |
216 |
| - | |
217 |
| - | |
218 |
| - | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
219 | 218 |
| |
220 | 219 |
| |
221 |
| - | |
222 |
| - | |
223 |
| - | |
224 |
| - | |
| 220 | + | |
225 | 221 |
| |
226 | 222 |
| |
227 | 223 |
| |
|
0 commit comments
Comments
(0)