Hi,
On 2023-06-27 15:33:30 +0900, Michael Paquier wrote:
> + /*
> + * Now that pg_class has been updated with its relevant information for
> + * the swap, update the dependency of the relation to point to its new
> + * table AM, if it has changed.
> + */
> + if (relam1 != relam2)
> + {
> + changeDependencyFor(RelationRelationId,
> + r1,
> + AccessMethodRelationId,
> + relam1,
> + relam2);
> + }
Should we check changeDependencyFor()'s return value? On a first glance it
looks like it would be an error to return 0 in this case?
Greetings,
Andres Freund