Re: GUC flags

On Wed, Dec 01, 2021 at 01:59:05AM -0600, Justin Pryzby wrote:
> On Tue, Nov 30, 2021 at 03:36:45PM +0900, Michael Paquier wrote:
>> -            gettext_noop("Waits N seconds on connection startup before authentication."),
>> +            gettext_noop("Sets the amount of seconds to wait on connection "
>> +                         "startup before authentication."),
>
> same

Thanks.  This makes things more consistent.

>>      {
>>          {"checkpoint_warning", PGC_SIGHUP, WAL_CHECKPOINTS,
>> -            gettext_noop("Enables warnings if checkpoint segments are filled more "
>> -                         "frequently than this."),
>> +            gettext_noop("Sets the maximum time before warning if checkpoints "
>> +                         "triggered by WAL volume happen too frequently."),
>>              gettext_noop("Write a message to the server log if checkpoints "
>> -                         "caused by the filling of checkpoint segment files happens more "
>> +                         "caused by the filling of WAL segment files happens more "
>
> It should say "happen" , since it's referring to "checkpoints".
> That was a pre-existing issue.

Indeed.

>>          {"log_parameter_max_length", PGC_SUSET, LOGGING_WHAT,
>> -            gettext_noop("When logging statements, limit logged parameter values to first N bytes."),
>> +            gettext_noop("Sets the maximum amount of data logged for bind "
>> +                         "parameter values when logging statements."),
>
> I think this one should actually say "in bytes" or at least say "maximum
> length".  It seems unlikely that someone is going to specify this in other
> units, and it's confusing to everyone else to refer to "amount of data" instead
> of "length in bytes".

Okay.  Do you like the updated version attached?

>> -            gettext_noop("Automatic log file rotation will occur after N minutes."),
>> +            gettext_noop("Sets the maximum amount of time to wait before "
>> +                         "forcing log file rotation."),
>
> Should it say "maximum" ?  Does that mean anything ?

To be consistent with the rest of your suggestions, we could use here:
"Sets the amount of time to wait before forcing log file rotation"

Thanks,
--
Michael