forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit2fe9777
committed
YA attempt to stabilize the results of the postgres_fdw regression test.
We've made multiple attempts to stabilize the plans shown by commit1bc0100, with little success so far. The reason for the remaininginstability seems to be that if a transaction (such as auto-analyze)is running concurrently with the test, then get_actual_variable_range mayreturn a maximum value for "T 1"."C 1" that's far away from the actual max,as a result of our having transiently inserted such a value earlier inthe test. Because we use a non-MVCC snapshot to fetch the value (forperformance reasons), the presence of other transactions can cause thatfunction to return entries that are actually dead.To fix, use a less extreme value in the earlier transient insertion, sothat whether it is visible or not won't affect the selectivity estimate.The use of 9999 there seems to have been picked with the aid of adartboard anyway, rather than having a specific reason.Discussion:https://postgr.es/m/16962.1523551784@sss.pgh.pa.us1 parenta4d56f5 commit2fe9777
File tree
2 files changed
+15
-15
lines changed- contrib/postgres_fdw
- expected
- sql
2 files changed
+15
-15
lines changedLines changed: 9 additions & 9 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
5367 | 5367 |
| |
5368 | 5368 |
| |
5369 | 5369 |
| |
5370 |
| - | |
| 5370 | + | |
5371 | 5371 |
| |
5372 | 5372 |
| |
5373 | 5373 |
| |
5374 | 5374 |
| |
5375 | 5375 |
| |
5376 | 5376 |
| |
5377 |
| - | |
| 5377 | + | |
5378 | 5378 |
| |
5379 | 5379 |
| |
5380 |
| - | |
| 5380 | + | |
5381 | 5381 |
| |
5382 | 5382 |
| |
5383 | 5383 |
| |
5384 | 5384 |
| |
5385 | 5385 |
| |
5386 | 5386 |
| |
5387 |
| - | |
| 5387 | + | |
5388 | 5388 |
| |
5389 | 5389 |
| |
5390 | 5390 |
| |
5391 | 5391 |
| |
5392 | 5392 |
| |
5393 |
| - | |
| 5393 | + | |
5394 | 5394 |
| |
5395 | 5395 |
| |
5396 |
| - | |
| 5396 | + | |
5397 | 5397 |
| |
5398 | 5398 |
| |
5399 | 5399 |
| |
5400 | 5400 |
| |
5401 | 5401 |
| |
5402 | 5402 |
| |
5403 |
| - | |
| 5403 | + | |
5404 | 5404 |
| |
5405 | 5405 |
| |
5406 | 5406 |
| |
5407 | 5407 |
| |
5408 | 5408 |
| |
5409 |
| - | |
| 5409 | + | |
5410 | 5410 |
| |
5411 | 5411 |
| |
5412 |
| - | |
| 5412 | + | |
5413 | 5413 |
| |
5414 | 5414 |
| |
5415 | 5415 |
| |
|
Lines changed: 6 additions & 6 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1099 | 1099 |
| |
1100 | 1100 |
| |
1101 | 1101 |
| |
1102 |
| - | |
1103 |
| - | |
| 1102 | + | |
| 1103 | + | |
1104 | 1104 |
| |
1105 |
| - | |
1106 |
| - | |
| 1105 | + | |
| 1106 | + | |
1107 | 1107 |
| |
1108 |
| - | |
1109 |
| - | |
| 1108 | + | |
| 1109 | + | |
1110 | 1110 |
| |
1111 | 1111 |
| |
1112 | 1112 |
| |
|
0 commit comments
Comments
(0)