Hi, Pavel! On Tue, Jun 16, 2026 at 12:57 AM Pavel Borisov <[email protected]> wrote: > I looked into the patch. It's simple enough and looks good to me. > I have one question: is it correct to pass 0 reloptions to > NewRelationCreateToastTable or we need to use transformRelOptions + > heap_reloptions pair to derive toast_reloptions as it is done in > ProcessUtilitySlow() and create_ctas_internal()?
Yes, (Datum) 0 is correct here. MERGE/SPLIT PARTITION grammar provides no syntactic way for the user to specify reloptions for the new partitions, so there is nothing to extract via transformRelOptions. This matches createPartitionTable() itself, which already calls heap_create_with_catalog(... reloptions=(Datum) 0 ...) for the main relation a few lines above; the TOAST table just inherits that choice. Extending the syntax to accept WITH (...) on new partitions would be a separate, larger feature. Out of scope for this bug fix. ------ Regards, Alexander Korotkov Supabase
