Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lutLT-0005yV-Ti for pgsql-hackers@arkaria.postgresql.org; Sun, 20 Jun 2021 09:02:23 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1lutLP-0000dr-MK for pgsql-hackers@arkaria.postgresql.org; Sun, 20 Jun 2021 09:02:19 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lutLP-0000di-Ef for pgsql-hackers@lists.postgresql.org; Sun, 20 Jun 2021 09:02:19 +0000 Received: from sfr-1.cri.mines-paristech.fr ([77.158.180.225] helo=smtp.cri.ensmp.fr) by magus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lutLM-0006SM-NN for pgsql-hackers@postgresql.org; Sun, 20 Jun 2021 09:02:18 +0000 Received: from pseudo (185.141.6.84.rev.sfr.net [84.6.141.185]) by smtp.cri.ensmp.fr (Postfix) with ESMTPSA id A353CC0285; Sun, 20 Jun 2021 11:04:10 +0200 (CEST) Date: Sun, 20 Jun 2021 11:02:14 +0200 (CEST) From: Fabien COELHO X-X-Sender: fabien@pseudo To: Thomas Munro cc: Alvaro Herrera , Michael Paquier , Gregory Smith , PostgreSQL-development , david.christensen@crunchydata.com Subject: Re: pgbench logging broken by time logic changes In-Reply-To: Message-ID: References: <202106200318.gxnon4vrc7t6@alvherre.pgsql> User-Agent: Alpine 2.22 (DEB 394 2020-01-19) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk > Upon reflection, that amounts to the same thing really, so yeah, > scratch that plan. I'll defer until after that (and then I'll be > leaning more towards the revert option). Sigh. I do not understand anything about the decision processus. If you do revert, please consider NOT reverting the tps computation changes intermixed in the patch. -- Fabien.