diff options
| author | Andres Freund <andres@anarazel.de> | 2019-05-14 12:11:26 -0700 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2019-05-14 12:19:32 -0700 |
| commit | aa4b8c61d2cd57b53be03defb04d59b232a0e150 (patch) | |
| tree | 3b7b1ca7702d20a11c652e876e666489f2d84996 /src/tutorial/funcs.c | |
| parent | 08e2edc0767ab6e619970f165cb34d4673105f23 (diff) | |
| download | postgresql-aa4b8c61d2cd57b53be03defb04d59b232a0e150.tar.gz postgresql-aa4b8c61d2cd57b53be03defb04d59b232a0e150.zip | |
Handle table_complete_speculative's succeeded argument as documented.
For some reason both callsite and the implementation for heapam had
the meaning inverted (i.e. succeeded == true was passed in case of
conflict). That's confusing.
I (Andres) briefly pondered whether it'd be better to rename
table_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 Linnakangas
Discussion:
https://postgr.es/m/CALfoeitk7-TACwYv3hCw45FNPjkA86RfXg4iQ5kAOPhR+F1Y4w@mail.gmail.com
https://postgr.es/m/97673451-339f-b21e-a781-998d06b1067c@iki.fi
Diffstat (limited to 'src/tutorial/funcs.c')
0 files changed, 0 insertions, 0 deletions