On 07/31/2018 12:45 PM, Bruce Momjian wrote:

Hi!,

Thanks for reviewing, I’ve updated the patch with the above mentioned incorrect
linkends as well as fixed the comments you made in a previous review.

The CF-builder-bot is red, but it’s because it’s trying to apply the already
committed patch which is in the attached datallowconn thread.
I think checksumhelper_cost_delay should be checksum_helper_cost_delay.
                                                     ^

Is "helper" the right word?

Based on other terminology within the postgresql.conf should it be "checksum_worker_cost_delay"?

JD



--
Command Prompt, Inc. || http://the.postgres.company/ || @cmdpromptinc
***  A fault and talent of mine is to tell it exactly how it is.  ***
PostgreSQL centered full stack support, consulting and development.
Advocate: @amplifypostgres || Learn: https://postgresconf.org
*****     Unless otherwise stated, opinions are my own.   *****


Reply via email to