- Notifications
You must be signed in to change notification settings - Fork67
Commit16810d9
committed
Revisit pathman readiness again.
Most pathman functions break if pathman is disabled: earlier I put defensechecks in split and merge range partitions, now it isdrop_range_partition_expand_next. Looks like the reason is that pathman cachesare not being invalidated if it is disabled: pathman_relcache_hook exits rightaway then. This is kinda reasonable: if we want to disable pathman completely,why maintain the caches? So this time try to bury the readiness check deeper,in get_pathman_relation_info itself.BTW, pathman caches are not dropped when it is disabled, which looks suspiciouson its own -- probably if we re-enable it later, caches might be inconsistent.1 parentd1032b0 commit16810d9
File tree
3 files changed
+4
-7
lines changed- expected
- src
3 files changed
+4
-7
lines changedLines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1009 | 1009 |
| |
1010 | 1010 |
| |
1011 | 1011 |
| |
1012 |
| - | |
| 1012 | + | |
1013 | 1013 |
| |
1014 | 1014 |
| |
1015 | 1015 |
| |
|
Lines changed: 0 additions & 6 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
489 | 489 |
| |
490 | 490 |
| |
491 | 491 |
| |
492 |
| - | |
493 |
| - | |
494 |
| - | |
495 | 492 |
| |
496 | 493 |
| |
497 | 494 |
| |
| |||
655 | 652 |
| |
656 | 653 |
| |
657 | 654 |
| |
658 |
| - | |
659 |
| - | |
660 |
| - | |
661 | 655 |
| |
662 | 656 |
| |
663 | 657 |
| |
|
Lines changed: 3 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
320 | 320 |
| |
321 | 321 |
| |
322 | 322 |
| |
| 323 | + | |
| 324 | + | |
| 325 | + | |
323 | 326 |
| |
324 | 327 |
| |
325 | 328 |
| |
|
0 commit comments
Comments
(0)