On 2018/06/15 12:29, Peter Eisentraut wrote:
> On 6/12/18 09:21, PG Bug reporting form wrote:
>> If a foreign table has a sequence attached (e.g. if the foreign table has a
>> "serial" column), and the foreign table owner is updated, the sequence owner
>> is not updated, leading to errors on restore:
>
> Yup, it only recurses to sequences for regular tables and materialized
> views. I suggest to remove the relkind check altogether. See attached
> patch.
>
> Can materialized views even have serial columns? I suspect materialized
> views entered this conditional because of the toast table check nearby.
>
> Also, can partitioned tables have serial columns? Are there more omissions?
>
> So fewer relkind enumerations are probably better.
+1 to recursing without a relkind check.
Thanks,
Amit