forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitaa4b8c6
committed
Handle table_complete_speculative's succeeded argument as documented.
For some reason both callsite and the implementation for heapam hadthe meaning inverted (i.e. succeeded == true was passed in case ofconflict). That's confusing.I (Andres) briefly pondered whether it'd be better to renametable_complete_speculative's argument to 'bool specConflict' or such,but decided not to. The 'complete' in the function name for me makes`succeeded` sound a bit better.Reported-By: Ashwin Agrawal, Melanie Plageman, Heikki LinnakangasDiscussion:https://postgr.es/m/CALfoeitk7-TACwYv3hCw45FNPjkA86RfXg4iQ5kAOPhR+F1Y4w@mail.gmail.comhttps://postgr.es/m/97673451-339f-b21e-a781-998d06b1067c@iki.fi1 parent08e2edc commitaa4b8c6
File tree
2 files changed
+2
-2
lines changed- src/backend
- access/heap
- executor
2 files changed
+2
-2
lines changedLines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
282 | 282 |
| |
283 | 283 |
| |
284 | 284 |
| |
285 |
| - | |
| 285 | + | |
286 | 286 |
| |
287 | 287 |
| |
288 | 288 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
556 | 556 |
| |
557 | 557 |
| |
558 | 558 |
| |
559 |
| - | |
| 559 | + | |
560 | 560 |
| |
561 | 561 |
| |
562 | 562 |
| |
|
0 commit comments
Comments
(0)