forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit9cb8409
committed
Repair problems occurring when multiple RI updates have to be done to the same
row within one query: we were firing check triggers before all the updateswere done, leading to bogus failures. Fix by making the triggers queued byan RI update go at the end of the outer query's trigger event list, therebyeffectively making the processing "breadth-first". This was indeed how itworked pre-8.0, so the bug does not occur in the 7.x branches.Per report from Pavel Stehule.1 parent5ff95e6 commit9cb8409
File tree
6 files changed
+170
-42
lines changed- src
- backend
- commands
- executor
- utils/adt
- include/executor
- test/regress
- expected
- sql
6 files changed
+170
-42
lines changedLines changed: 32 additions & 21 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
7 | 7 |
| |
8 | 8 |
| |
9 | 9 |
| |
10 |
| - | |
| 10 | + | |
11 | 11 |
| |
12 | 12 |
| |
13 | 13 |
| |
| |||
2332 | 2332 |
| |
2333 | 2333 |
| |
2334 | 2334 |
| |
| 2335 | + | |
2335 | 2336 |
| |
2336 | 2337 |
| |
2337 | 2338 |
| |
| |||
2364 | 2365 |
| |
2365 | 2366 |
| |
2366 | 2367 |
| |
| 2368 | + | |
| 2369 | + | |
| 2370 | + | |
| 2371 | + | |
| 2372 | + | |
| 2373 | + | |
| 2374 | + | |
| 2375 | + | |
| 2376 | + | |
| 2377 | + | |
| 2378 | + | |
| 2379 | + | |
| 2380 | + | |
2367 | 2381 |
| |
2368 | 2382 |
| |
2369 | 2383 |
| |
| |||
2375 | 2389 |
| |
2376 | 2390 |
| |
2377 | 2391 |
| |
| 2392 | + | |
| 2393 | + | |
| 2394 | + | |
| 2395 | + | |
| 2396 | + | |
| 2397 | + | |
2378 | 2398 |
| |
| 2399 | + | |
2379 | 2400 |
| |
2380 | 2401 |
| |
2381 | 2402 |
| |
2382 |
| - | |
2383 |
| - | |
2384 |
| - | |
2385 |
| - | |
2386 |
| - | |
2387 |
| - | |
2388 |
| - | |
2389 | 2403 |
| |
2390 |
| - | |
| 2404 | + | |
| 2405 | + | |
2391 | 2406 |
| |
2392 |
| - | |
2393 |
| - | |
2394 |
| - | |
2395 |
| - | |
2396 |
| - | |
2397 |
| - | |
2398 |
| - | |
2399 |
| - | |
| 2407 | + | |
2400 | 2408 |
| |
2401 | 2409 |
| |
2402 | 2410 |
| |
| |||
2421 | 2429 |
| |
2422 | 2430 |
| |
2423 | 2431 |
| |
| 2432 | + | |
2424 | 2433 |
| |
2425 | 2434 |
| |
2426 | 2435 |
| |
| |||
2471 | 2480 |
| |
2472 | 2481 |
| |
2473 | 2482 |
| |
2474 |
| - | |
| 2483 | + | |
2475 | 2484 |
| |
2476 | 2485 |
| |
2477 | 2486 |
| |
| |||
2600 | 2609 |
| |
2601 | 2610 |
| |
2602 | 2611 |
| |
| 2612 | + | |
| 2613 | + | |
2603 | 2614 |
| |
2604 | 2615 |
| |
2605 | 2616 |
| |
2606 | 2617 |
| |
2607 |
| - | |
| 2618 | + | |
2608 | 2619 |
| |
2609 | 2620 |
| |
2610 | 2621 |
| |
| |||
2648 | 2659 |
| |
2649 | 2660 |
| |
2650 | 2661 |
| |
2651 |
| - | |
| 2662 | + | |
2652 | 2663 |
| |
2653 | 2664 |
| |
2654 | 2665 |
| |
| |||
3211 | 3222 |
| |
3212 | 3223 |
| |
3213 | 3224 |
| |
3214 |
| - | |
| 3225 | + | |
3215 | 3226 |
| |
3216 | 3227 |
| |
3217 | 3228 |
| |
|
Lines changed: 23 additions & 15 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
8 | 8 |
| |
9 | 9 |
| |
10 | 10 |
| |
11 |
| - | |
| 11 | + | |
12 | 12 |
| |
13 | 13 |
| |
14 | 14 |
| |
| |||
39 | 39 |
| |
40 | 40 |
| |
41 | 41 |
| |
42 |
| - | |
| 42 | + | |
43 | 43 |
| |
44 |
| - | |
| 44 | + | |
45 | 45 |
| |
46 | 46 |
| |
47 | 47 |
| |
| |||
316 | 316 |
| |
317 | 317 |
| |
318 | 318 |
| |
319 |
| - | |
| 319 | + | |
320 | 320 |
| |
321 | 321 |
| |
322 | 322 |
| |
| |||
349 | 349 |
| |
350 | 350 |
| |
351 | 351 |
| |
352 |
| - | |
| 352 | + | |
353 | 353 |
| |
354 | 354 |
| |
355 | 355 |
| |
| |||
364 | 364 |
| |
365 | 365 |
| |
366 | 366 |
| |
367 |
| - | |
368 |
| - | |
369 |
| - | |
| 367 | + | |
| 368 | + | |
| 369 | + | |
| 370 | + | |
| 371 | + | |
| 372 | + | |
370 | 373 |
| |
371 | 374 |
| |
372 | 375 |
| |
| |||
375 | 378 |
| |
376 | 379 |
| |
377 | 380 |
| |
378 |
| - | |
| 381 | + | |
379 | 382 |
| |
380 | 383 |
| |
381 | 384 |
| |
| |||
392 | 395 |
| |
393 | 396 |
| |
394 | 397 |
| |
395 |
| - | |
| 398 | + | |
396 | 399 |
| |
397 | 400 |
| |
398 | 401 |
| |
| |||
1428 | 1431 |
| |
1429 | 1432 |
| |
1430 | 1433 |
| |
| 1434 | + | |
| 1435 | + | |
1431 | 1436 |
| |
1432 | 1437 |
| |
1433 | 1438 |
| |
1434 | 1439 |
| |
1435 | 1440 |
| |
1436 |
| - | |
| 1441 | + | |
1437 | 1442 |
| |
1438 | 1443 |
| |
1439 | 1444 |
| |
| |||
1589 | 1594 |
| |
1590 | 1595 |
| |
1591 | 1596 |
| |
1592 |
| - | |
| 1597 | + | |
| 1598 | + | |
1593 | 1599 |
| |
1594 | 1600 |
| |
1595 | 1601 |
| |
| |||
1680 | 1686 |
| |
1681 | 1687 |
| |
1682 | 1688 |
| |
1683 |
| - | |
| 1689 | + | |
1684 | 1690 |
| |
1685 | 1691 |
| |
1686 | 1692 |
| |
| |||
1726 | 1732 |
| |
1727 | 1733 |
| |
1728 | 1734 |
| |
1729 |
| - | |
| 1735 | + | |
| 1736 | + | |
1730 | 1737 |
| |
1731 | 1738 |
| |
1732 | 1739 |
| |
| |||
1743 | 1750 |
| |
1744 | 1751 |
| |
1745 | 1752 |
| |
1746 |
| - | |
| 1753 | + | |
| 1754 | + | |
1747 | 1755 |
| |
1748 | 1756 |
| |
1749 | 1757 |
| |
|
Lines changed: 3 additions & 3 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
15 | 15 |
| |
16 | 16 |
| |
17 | 17 |
| |
18 |
| - | |
| 18 | + | |
19 | 19 |
| |
20 | 20 |
| |
21 | 21 |
| |
| |||
2774 | 2774 |
| |
2775 | 2775 |
| |
2776 | 2776 |
| |
2777 |
| - | |
| 2777 | + | |
2778 | 2778 |
| |
2779 | 2779 |
| |
2780 | 2780 |
| |
| |||
3308 | 3308 |
| |
3309 | 3309 |
| |
3310 | 3310 |
| |
3311 |
| - | |
| 3311 | + | |
3312 | 3312 |
| |
3313 | 3313 |
| |
3314 | 3314 |
| |
|
Lines changed: 2 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
6 | 6 |
| |
7 | 7 |
| |
8 | 8 |
| |
9 |
| - | |
| 9 | + | |
10 | 10 |
| |
11 | 11 |
| |
12 | 12 |
| |
| |||
104 | 104 |
| |
105 | 105 |
| |
106 | 106 |
| |
107 |
| - | |
| 107 | + | |
108 | 108 |
| |
109 | 109 |
| |
110 | 110 |
| |
|
Lines changed: 68 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
646 | 646 |
| |
647 | 647 |
| |
648 | 648 |
| |
649 |
| - | |
650 | 649 |
| |
651 | 650 |
| |
652 | 651 |
| |
| |||
1230 | 1229 |
| |
1231 | 1230 |
| |
1232 | 1231 |
| |
| 1232 | + | |
| 1233 | + | |
| 1234 | + | |
| 1235 | + | |
| 1236 | + | |
| 1237 | + | |
| 1238 | + | |
| 1239 | + | |
| 1240 | + | |
| 1241 | + | |
| 1242 | + | |
| 1243 | + | |
| 1244 | + | |
| 1245 | + | |
| 1246 | + | |
| 1247 | + | |
| 1248 | + | |
| 1249 | + | |
| 1250 | + | |
| 1251 | + | |
| 1252 | + | |
| 1253 | + | |
| 1254 | + | |
| 1255 | + | |
| 1256 | + | |
| 1257 | + | |
| 1258 | + | |
| 1259 | + | |
| 1260 | + | |
| 1261 | + | |
| 1262 | + | |
| 1263 | + | |
| 1264 | + | |
| 1265 | + | |
| 1266 | + | |
| 1267 | + | |
| 1268 | + | |
| 1269 | + | |
| 1270 | + | |
| 1271 | + | |
| 1272 | + | |
| 1273 | + | |
| 1274 | + | |
| 1275 | + | |
| 1276 | + | |
| 1277 | + | |
| 1278 | + | |
| 1279 | + | |
| 1280 | + | |
| 1281 | + | |
| 1282 | + | |
| 1283 | + | |
| 1284 | + | |
| 1285 | + | |
| 1286 | + | |
| 1287 | + | |
| 1288 | + | |
| 1289 | + | |
| 1290 | + | |
| 1291 | + | |
| 1292 | + | |
| 1293 | + | |
| 1294 | + | |
| 1295 | + | |
| 1296 | + | |
| 1297 | + | |
| 1298 | + | |
| 1299 | + |
0 commit comments
Comments
(0)