forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit3e8235b
committed
Fix multiranges to behave more like dependent types.
For most purposes, multiranges act like dependent objects of theassociated range type: you can't create them separately or drop themseparately. This is like the way that autogenerated array typesbehave. However, a couple of points were overlooked: array typesautomatically track the ownership of their base type, and array typesdo not have their own permissions but use those of the base type,while multiranges didn't emulate those behaviors. This is fairlybroken, mainly because pg_dump doesn't think it needs to worry aboutmultiranges as separate objects, and thus it fails to dump/restoreownership or permissions of multiranges.There's no apparent value in letting a multirange diverge fromits parent's ownership or permissions, so let's make them act likearrays in these respects. However, we continue to let multirangesbe renamed or moved to a different schema independently of theirparent, since that doesn't break anything.Discussion:https://postgr.es/m/1580383.1705343264@sss.pgh.pa.us1 parentbd8fc16 commit3e8235b
File tree
7 files changed
+155
-23
lines changed- src
- backend
- catalog
- commands
- bin/pg_dump
- test/regress
- expected
- sql
7 files changed
+155
-23
lines changedLines changed: 35 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2447 | 2447 |
| |
2448 | 2448 |
| |
2449 | 2449 |
| |
| 2450 | + | |
2450 | 2451 |
| |
2451 | 2452 |
| |
2452 | 2453 |
| |
2453 | 2454 |
| |
2454 | 2455 |
| |
| 2456 | + | |
| 2457 | + | |
| 2458 | + | |
| 2459 | + | |
| 2460 | + | |
2455 | 2461 |
| |
2456 | 2462 |
| |
2457 | 2463 |
| |
| |||
3806 | 3812 |
| |
3807 | 3813 |
| |
3808 | 3814 |
| |
| 3815 | + | |
| 3816 | + | |
| 3817 | + | |
| 3818 | + | |
| 3819 | + | |
| 3820 | + | |
| 3821 | + | |
| 3822 | + | |
| 3823 | + | |
| 3824 | + | |
| 3825 | + | |
| 3826 | + | |
| 3827 | + | |
| 3828 | + | |
| 3829 | + | |
| 3830 | + | |
| 3831 | + | |
| 3832 | + | |
| 3833 | + | |
| 3834 | + | |
| 3835 | + | |
| 3836 | + | |
| 3837 | + | |
| 3838 | + | |
| 3839 | + | |
| 3840 | + | |
| 3841 | + | |
| 3842 | + | |
| 3843 | + | |
3809 | 3844 |
| |
3810 | 3845 |
| |
3811 | 3846 |
| |
|
Lines changed: 29 additions & 12 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
326 | 326 |
| |
327 | 327 |
| |
328 | 328 |
| |
329 |
| - | |
330 |
| - | |
331 |
| - | |
332 |
| - | |
333 |
| - | |
334 |
| - | |
| 329 | + | |
| 330 | + | |
| 331 | + | |
| 332 | + | |
| 333 | + | |
| 334 | + | |
335 | 335 |
| |
336 | 336 |
| |
| 337 | + | |
337 | 338 |
| |
338 | 339 |
| |
339 | 340 |
| |
| |||
534 | 535 |
| |
535 | 536 |
| |
536 | 537 |
| |
537 |
| - | |
538 |
| - | |
| 538 | + | |
| 539 | + | |
| 540 | + | |
539 | 541 |
| |
540 | 542 |
| |
541 | 543 |
| |
| |||
601 | 603 |
| |
602 | 604 |
| |
603 | 605 |
| |
604 |
| - | |
| 606 | + | |
| 607 | + | |
| 608 | + | |
605 | 609 |
| |
606 | 610 |
| |
607 |
| - | |
| 611 | + | |
608 | 612 |
| |
609 | 613 |
| |
610 |
| - | |
| 614 | + | |
611 | 615 |
| |
612 | 616 |
| |
613 | 617 |
| |
614 |
| - | |
| 618 | + | |
| 619 | + | |
615 | 620 |
| |
| 621 | + | |
| 622 | + | |
616 | 623 |
| |
617 | 624 |
| |
618 | 625 |
| |
| |||
727 | 734 |
| |
728 | 735 |
| |
729 | 736 |
| |
| 737 | + | |
| 738 | + | |
| 739 | + | |
| 740 | + | |
| 741 | + | |
| 742 | + | |
| 743 | + | |
| 744 | + | |
| 745 | + | |
| 746 | + | |
730 | 747 |
| |
731 | 748 |
| |
732 | 749 |
| |
|
Lines changed: 36 additions & 6 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
3647 | 3647 |
| |
3648 | 3648 |
| |
3649 | 3649 |
| |
| 3650 | + | |
| 3651 | + | |
3650 | 3652 |
| |
3651 | 3653 |
| |
3652 | 3654 |
| |
| |||
3730 | 3732 |
| |
3731 | 3733 |
| |
3732 | 3734 |
| |
| 3735 | + | |
| 3736 | + | |
| 3737 | + | |
| 3738 | + | |
| 3739 | + | |
| 3740 | + | |
| 3741 | + | |
| 3742 | + | |
| 3743 | + | |
| 3744 | + | |
| 3745 | + | |
| 3746 | + | |
| 3747 | + | |
| 3748 | + | |
| 3749 | + | |
3733 | 3750 |
| |
3734 | 3751 |
| |
3735 | 3752 |
| |
| |||
3769 | 3786 |
| |
3770 | 3787 |
| |
3771 | 3788 |
| |
3772 |
| - | |
3773 |
| - | |
3774 |
| - | |
3775 |
| - | |
| 3789 | + | |
| 3790 | + | |
| 3791 | + | |
| 3792 | + | |
3776 | 3793 |
| |
3777 | 3794 |
| |
3778 |
| - | |
| 3795 | + | |
3779 | 3796 |
| |
3780 | 3797 |
| |
3781 | 3798 |
| |
| |||
3815 | 3832 |
| |
3816 | 3833 |
| |
3817 | 3834 |
| |
3818 |
| - | |
| 3835 | + | |
3819 | 3836 |
| |
3820 | 3837 |
| |
3821 | 3838 |
| |
| |||
3865 | 3882 |
| |
3866 | 3883 |
| |
3867 | 3884 |
| |
| 3885 | + | |
| 3886 | + | |
| 3887 | + | |
| 3888 | + | |
| 3889 | + | |
| 3890 | + | |
| 3891 | + | |
| 3892 | + | |
| 3893 | + | |
| 3894 | + | |
| 3895 | + | |
| 3896 | + | |
| 3897 | + | |
3868 | 3898 |
| |
3869 | 3899 |
| |
3870 | 3900 |
| |
|
Lines changed: 4 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1868 | 1868 |
| |
1869 | 1869 |
| |
1870 | 1870 |
| |
1871 |
| - | |
| 1871 | + | |
1872 | 1872 |
| |
1873 | 1873 |
| |
1874 | 1874 |
| |
1875 | 1875 |
| |
1876 | 1876 |
| |
1877 | 1877 |
| |
1878 | 1878 |
| |
1879 |
| - | |
1880 |
| - | |
| 1879 | + | |
| 1880 | + | |
1881 | 1881 |
| |
1882 | 1882 |
| |
1883 | 1883 |
| |
| |||
5770 | 5770 |
| |
5771 | 5771 |
| |
5772 | 5772 |
| |
5773 |
| - | |
| 5773 | + | |
5774 | 5774 |
| |
5775 | 5775 |
| |
5776 | 5776 |
| |
|
Lines changed: 0 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
144 | 144 |
| |
145 | 145 |
| |
146 | 146 |
| |
147 |
| - | |
148 | 147 |
| |
149 | 148 |
| |
150 | 149 |
| |
|
Lines changed: 30 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
3115 | 3115 |
| |
3116 | 3116 |
| |
3117 | 3117 |
| |
| 3118 | + | |
| 3119 | + | |
| 3120 | + | |
| 3121 | + | |
| 3122 | + | |
| 3123 | + | |
| 3124 | + | |
| 3125 | + | |
| 3126 | + | |
| 3127 | + | |
| 3128 | + | |
| 3129 | + | |
| 3130 | + | |
| 3131 | + | |
| 3132 | + | |
| 3133 | + | |
| 3134 | + | |
| 3135 | + | |
| 3136 | + | |
| 3137 | + | |
| 3138 | + | |
| 3139 | + | |
| 3140 | + | |
| 3141 | + | |
| 3142 | + | |
| 3143 | + | |
| 3144 | + | |
| 3145 | + | |
| 3146 | + | |
| 3147 | + | |
3118 | 3148 |
| |
3119 | 3149 |
| |
3120 | 3150 |
| |
|
Lines changed: 21 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
700 | 700 |
| |
701 | 701 |
| |
702 | 702 |
| |
| 703 | + | |
| 704 | + | |
| 705 | + | |
| 706 | + | |
| 707 | + | |
| 708 | + | |
| 709 | + | |
| 710 | + | |
| 711 | + | |
| 712 | + | |
| 713 | + | |
| 714 | + | |
| 715 | + | |
| 716 | + | |
| 717 | + | |
| 718 | + | |
| 719 | + | |
| 720 | + | |
| 721 | + | |
| 722 | + | |
| 723 | + | |
703 | 724 |
| |
704 | 725 |
| |
705 | 726 |
| |
|
0 commit comments
Comments
(0)