forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit543e205
committed
Fix
Commite5550d5 added some newtests for ALTER TABLE which involved table scans. Whendefault_transaction_isolation = 'serializable' these acquirerelation-level SIReadLocks. The test results didn't cope withthat. Add SIReadLock as the minimum lock level for purposes ofthese tests.This could also be fixed by excluding this type of lock from themy_locks view, but it would be a bug for SIReadLock to show up fora relation which was not otherwise locked, so do it this way toallow that sort of condition to cause a regression test failure.There is some question whether we could avoid taking SIReadLocksduring these operations, but confirming the safety of that andfiguring out how to avoid the locks is not trivial, and would bea separate patch.Backpatch to 9.4 where the new tests were added.make installcheck
for serializable transactions.1 parent4d94b5f commit543e205
2 files changed
+4
-2
lines changedLines changed: 2 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1837 | 1837 |
| |
1838 | 1838 |
| |
1839 | 1839 |
| |
1840 |
| - | |
| 1840 | + | |
| 1841 | + | |
1841 | 1842 |
| |
1842 | 1843 |
| |
1843 | 1844 |
| |
|
Lines changed: 2 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1270 | 1270 |
| |
1271 | 1271 |
| |
1272 | 1272 |
| |
1273 |
| - | |
| 1273 | + | |
| 1274 | + | |
1274 | 1275 |
| |
1275 | 1276 |
| |
1276 | 1277 |
| |
|
0 commit comments
Comments
(0)