Re: COPY TO (FREEZE)?

On Sat, Oct 28, 2023 at 09:54:05PM -0400, Tom Lane wrote:
> Bruce Momjian <bruce@momjian.us> writes:
> > On Sat, Oct 28, 2023 at 09:39:53PM -0400, Tom Lane wrote:
> >> Not thrilled by the wording here.
>
> > I think it is modeled after:
>
> >     errmsg("COPY force null only available using COPY FROM")));
>
> Well, now that you bring it up, that's no sterling example of
> clear writing either.  Maybe change that while we're at it,
> say to "FORCE NULL option must not be used in COPY TO"?

I used:

        "COPY FREEZE mode cannot be used with COPY FROM"

and adjusted the others.

> (Also, has it got the right ERRCODE?)

Fixed, and the other cases too.  Patch attached.

--
  Bruce Momjian  <bruce@momjian.us>        https://momjian.us
  EDB                                      https://enterprisedb.com

  Only you can decide what is important to you.

 errmsg("COPY force not null only available using COPY FROM")));

+ (errcode(ERRCODE_INVALID_PARAMETER_VALUE),

+ errmsg("COPY force not null cannot be used with COPY FROM")));