Re: ALTER FOREIGN DATA WRAPPER can drop dependency on handler

On Wed, Mar 18, 2026 at 12:09:54PM -0700, Jeff Davis wrote:
> --- a/src/backend/commands/foreigncmds.c
> +++ b/src/backend/commands/foreigncmds.c
> @@ -786,6 +786,11 @@ AlterForeignDataWrapper(ParseState *pstate, AlterFdwStmt *stmt)
>          ereport(WARNING,
>                  (errmsg("changing the foreign-data wrapper handler can change behavior of existing foreign
tables")));
>      }
> +    else
> +    {
> +        /* handler unchanged */
> +        fdwhandler = fdwForm->fdwhandler;
> +    }

This matches what we do for the validator a few lines down, so it makes
sense to me.  Could it be worth adding a test or two?

-- 
nathan