Re: Log a sample of transactions

Hello,

On 3/28/19 3:18 AM, Masahiko Sawada wrote:
>>>
>>> Sorry, I changed that, someone suggest using either "0" and "1", or
>>> "0.0" and "1.0" but not mixing both.
>
> Agreed with using "0.0" and "1.0".
>
>>> I will remove this change.
>>>
>
> --- a/src/backend/utils/misc/postgresql.conf.sample
> +++ b/src/backend/utils/misc/postgresql.conf.sample
> +#log_transaction_sample_rate = 0 # Fraction of transactions
> whose statements
> + # are logged regardless of
> their duration. 1.0 logs all
> + # statements from all
> transactions, 0 never logs.
>
> --- a/src/backend/utils/misc/guc.c
> +++ b/src/backend/utils/misc/guc.c
> + {"log_transaction_sample_rate", PGC_SUSET, LOGGING_WHEN,
> + gettext_noop("Set the fraction of transactions
> to log for new transactions."),
> + gettext_noop("Logs all statements from a
> fraction of transactions. "
> + "Use a value between
> 0 (never log) and 1 (log all "
> + "statements for all
> transactions).")
>
> I think it would be better to use the same number in both guc.c and
> postgresql.conf.sample. How about unifying them to "0.0" and "1.0" for
> consistency?
>

I changed to use both 0.0 and 1.0.

Thanks!