Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1t9RS0-00ACXm-V6 for pgsql-committers@arkaria.postgresql.org; Fri, 08 Nov 2024 16:03:08 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.94.2) (envelope-from ) id 1t9RRx-00Ar13-RL for pgsql-committers@arkaria.postgresql.org; Fri, 08 Nov 2024 16:03:06 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1t9RRx-00Ar0t-Jx for pgsql-committers@lists.postgresql.org; Fri, 08 Nov 2024 16:03:06 +0000 Received: from sss.pgh.pa.us ([68.162.161.243]) by magus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1t9RRt-000sBv-St for pgsql-committers@lists.postgresql.org; Fri, 08 Nov 2024 16:03:05 +0000 Received: from sss1.sss.pgh.pa.us (localhost [127.0.0.1]) by sss.pgh.pa.us (8.15.2/8.15.2) with ESMTP id 4A8G30l73647361; Fri, 8 Nov 2024 11:03:00 -0500 From: Tom Lane To: Robert Haas cc: Andrew Dunstan , pgsql-committers@lists.postgresql.org Subject: Re: pgsql: Fix our Windows stat() emulation to handle file sizes > 4GB. In-reply-to: References: Comments: In-reply-to Robert Haas message dated "Fri, 08 Nov 2024 08:45:27 -0500" MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-ID: <3647359.1731081780.1@sss.pgh.pa.us> Content-Transfer-Encoding: quoted-printable Date: Fri, 08 Nov 2024 11:03:00 -0500 Message-ID: <3647360.1731081780@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk Robert Haas writes: > On Thu, Nov 7, 2024 at 6:19=E2=80=AFPM Andrew Dunstan wrote: >> Juan Jos=C3=A9 Santamar=C3=ADa Flecha, reviewed by Emil Iggland; >> based on prior work by Michael Paquier, Sergey Zubkovsky, and others >> = >> Author: Alexandra Wang > This authorship information is confusing. Yes. Worse, there is no link to what prompted this sudden burst of back-patches of years-old commits. After digging around I guess it was this thread: https://www.postgresql.org/message-id/flat/CA%2BpA0kLc5VxOaO4WfLjmh7W0V%2B= quVvVtT5CaRVRAZMuh0zft4Q%40mail.gmail.com I'm nervous about pushing these in mere hours before a release freeze, primarily because there's no way to be sure whether any necessary follow-up fixes got missed. If we were unwilling to back-patch at the time, is reversing that decision such a good idea? I'm also confused about how to document them in the release notes. Alexandra should get some credit I guess for collecting and testing the patches, but she's not the original author(s). regards, tom lane