Re: Add hint message for check_log_destination()

On Fri, Jul 7, 2023 at 4:53 PM Japin Li <japinli(at)hotmail(dot)com> wrote:
>
>
> On Fri, 07 Jul 2023 at 14:46, jian he <jian(dot)universality(at)gmail(dot)com> wrote:
> > On Fri, Jul 7, 2023 at 1:06 PM Japin Li <japinli(at)hotmail(dot)com> wrote:
> >>
> >>
> >> Hi, hackers
> >>
> >> When I try to change log_destination using ALTER SYSTEM with the wrong value,
> >> it complains of the "Unrecognized key word" without available values. This
> >> patch tries to add a hint message that provides available values for
> >> log_destination. Any thoughts?

+1

+ appendStringInfo(&errhint, "\"stderr\"");
+#ifdef HAVE_SYSLOG
+ appendStringInfo(&errhint, ", \"syslog\"");
+#endif
+#ifdef WIN32
+ appendStringInfo(&errhint, ", \"eventlog\"");
+#endif
+ appendStringInfo(&errhint, ", \"csvlog\", and \"jsonlog\"");

I think using appendStringInfoString() is a bit more natural and faster.

Regards,

--
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com