Re: Online enabling of checksums
Bruce Momjian <bruce@momjian.us>
From: Bruce Momjian <bruce@momjian.us>
To: "Joshua D. Drake" <jd@commandprompt.com>
Cc: Daniel Gustafsson <daniel@yesql.se>, Sergei Kornilov <sk@zsrv.org>, PostgreSQL Developers <pgsql-hackers@lists.postgresql.org>, Magnus Hagander <magnus@hagander.net>
Date: 2018-07-31T20:43:16Z
Lists: pgsql-hackers
On Tue, Jul 31, 2018 at 12:52:40PM -0700, Joshua Drake wrote: > 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"? +1> -- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + As you are, so once was I. As I am, so you will be. + + Ancient Roman grave inscription +
Commits
-
Online enabling and disabling of data checksums
- f19c0eccae96 19 (unreleased) landed
-
Deactive flapping checksum isolation tests.
- bf75fe47e444 11.0 landed
-
Add support for coordinating record typmods among parallel workers.
- cc5f81366c36 11.0 cited