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