- Notifications
You must be signed in to change notification settings - Fork28
Commit28f4b61
committed
Reinstate HEAP_XMAX_LOCK_ONLY|HEAP_KEYS_UPDATED as allowed
Commit 866e24d47db1 added an assert that HEAP_XMAX_LOCK_ONLY andHEAP_KEYS_UPDATED cannot appear together, on the faulty assumption thatthe latter necessarily referred to an update and not a tuple lock; butthat's wrong, because SELECT FOR UPDATE can use precisely thatcombination, as evidenced by the amcheck test case added here.Remove the Assert(), and also patch amcheck's verify_heapam.c to notcomplain if the combination is found. Also, out of overabundance ofcaution, update (across all branches) README.tuplock to be more explicitabout this.Author: Julien Rouhaud <rjuju123@gmail.com>Reviewed-by: Mahendra Singh Thalor <mahi6run@gmail.com>Reviewed-by: Dilip Kumar <dilipbalaut@gmail.com>Discussion:https://postgr.es/m/20210124061758.GA11756@nol1 parent186f616 commit28f4b61
1 file changed
+4
-3
lines changedLines changed: 4 additions & 3 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
146 | 146 |
| |
147 | 147 |
| |
148 | 148 |
| |
149 |
| - | |
150 |
| - | |
151 |
| - | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
152 | 153 |
| |
153 | 154 |
| |
154 | 155 |
|
0 commit comments
Comments
(0)