On Thu, Jun 18, 2026 at 6:49 PM Justin Pryzby <pryzby@telsasoft.com> wrote:
> On Thu, Jun 18, 2026 at 10:31:11AM +0300, Alexander Korotkov wrote:
> > Pushed with your suggestions accepted.
>
> Thanks. When I went back to test this, I merged ~25 partitions that
> were all on the same tablespace, but the merged table was created on the
> default tablespace.
>
> I tried again with default_tablespace set, but it was ignored. I think
> that's wrong. It's good to follow the tablespace of the parent table,
> but if it has no tablespace set, default_tablespace should be obeyed.
> See surrounding logic in DefineRelation.
>
> I see the docs say this:
> + <command>ALTER TABLE MERGE PARTITION</command> uses the partitioned
> + table itself as the template to construct the new partition.
> + The new partition will inherit the same table access method, persistence
> + type, and tablespace as the partitioned table.
Correct, please see the attached patch, it makes
createPartitionTable() deal with tablespaces the same was as
DefineRelation() does.
------
Regards,
Alexander Korotkov
Supabase