- 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 changed| 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)