On Fri, 07 Jul 2023 at 16:21, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> wrote:
> 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.
>
Thanks for your review! Fixed as per your suggession.