On Fri, Dec 27, 2024 at 03:01:58PM +0800, jian he wrote: > please check attached for changes within ATPrepAlterColumnType
Sorry for the late reply. This is more complete. Two strings are
more complex after doing the coerce_to_target_type(), but honestly I
am not these are worth having a parser_errposition() as they also have
an errhint().
> Maybe we can change to
> errmsg("conflicting NO INHERIT declarations for not-null constraints
> on column \"%s\"", column->colname),
> errhint("specified primary key or identity sequence imply an inherited
> not-null constraint will be created")
>
> what do you think?
Hmm. Not sure that this is an improvement. Will think a bit more
about it.
--
Michael
signature.asc
Description: PGP signature
